Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add API_KEY support #740

Closed
wants to merge 1 commit into from
Closed

Conversation

bnusunny
Copy link
Contributor

Description

This PR fixes #729 and #480

This PR adds api-key flag. When this flag is set, all API requests needs to provide the same value as Bear token in the request hear. If this flag is not set, no auth will be enabled. This flag is not set by default.

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

@mudler
Copy link
Owner

mudler commented Jul 15, 2023

Thanks @bnusunny ! I'd be tempted to document around usage of https://github.com/Kong/kong and alikes rather then implementing something specific in the code.

However, I think this approach is simple and provide an easy way to setup for minimal configurations that might be enough. Just one small nit, naybe we can make api-keys a list? so can specify multiply tokens instead of one

@bnusunny
Copy link
Contributor Author

Sure. I will make it a list.

@neboman11 neboman11 mentioned this pull request Aug 9, 2023
1 task
@neboman11
Copy link
Contributor

This PR can be closed since #877 is merged.

@bnusunny bnusunny closed this Aug 14, 2023
@bnusunny bnusunny deleted the add_api_key_support branch August 14, 2023 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Protect endpoints with an API key
3 participants