-
Notifications
You must be signed in to change notification settings - Fork 142
Enhancement Proposal: Authentication Filter #4235
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
Open
shaun-nx
wants to merge
40
commits into
main
Choose a base branch
from
proposal/authentication-filter
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+1,132
−1
Open
Changes from 1 commit
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
d91a5f4
Add Implementable Proposal for AuthenticationFilter
shaun-nx 9047c3e
Update auth header code block
shaun-nx c937366
Fix pre-commit and lint errors
shaun-nx 1b8bac2
Update Golang API with defaults and CEL validation with kubebuilder
shaun-nx 2f143e9
Add additional defaults and CEL validations
shaun-nx 61f479c
Merge branch 'main' into proposal/authentication-filter
shaun-nx 47ff38b
Fix typos
shaun-nx 40b8224
Update comments in GolangAPI to decribe relative NGINX directives
shaun-nx 24966b8
Update API and Security Considerations for ReferenceGrant integration
shaun-nx ecceb93
Merge branch 'main' into proposal/authentication-filter
shaun-nx da1b17e
Fix pre-commit errors
shaun-nx 38dd8f7
Fix typos and grammer
shaun-nx e362745
Update BasicAuth AIP and examples to use `secretRef`
shaun-nx dd5aaa8
Update KeyCache to use v1alpha1.Duration
shaun-nx bf3ed2b
Merge branch 'main' into proposal/authentication-filter
shaun-nx eb49b32
Merge branch 'main' into proposal/authentication-filter
shaun-nx a86a3ae
Move kubebuilder validation, remove mountpath and configmap options, …
shaun-nx 79b957d
Update jwks_uri internal uri
shaun-nx e0ec4fb
Fix typos
shaun-nx d3ebed4
Fix comment indentation
shaun-nx 4f1c893
Adjust struct indentation
shaun-nx 5ff7389
Pre-commit check
shaun-nx dd24287
Ensure no default for JWT key mode
shaun-nx 14b84a9
Use SecretObjectReference for secretRef, remove references to ConfigM…
shaun-nx 8526c7f
Add references to NGINX directives
shaun-nx 93486eb
Remove `key` from `secretRef`
shaun-nx d91b389
Remove trailing whitespace
shaun-nx b6fb76b
Add additional comments
shaun-nx 73594af
Update comments and restructure api
shaun-nx 7aede4c
Update security details on headers
shaun-nx acbb54c
Merge branch 'main' into proposal/authentication-filter
shaun-nx 4aae8e7
Merge branch 'main' into proposal/authentication-filter
shaun-nx 1029c3e
Remove JWT key word from fields
shaun-nx c864630
Add default Real for basic auth
shaun-nx ef57f2a
Fix typo
shaun-nx 22d2726
Update Status section on using NGINX OSS with JWT auth
shaun-nx 9e6b3c9
Set optioanl JWT fields as stretch goals
shaun-nx 21b5611
Add stretch goals
shaun-nx 65c1adc
Update secret ref to use LocalObjectReferenceWithKey struct
shaun-nx a0c8c04
Fix typo and update validation section
shaun-nx File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not opposed to having JWT in the names of these fields. It makes it clear that they are for JWT. My only gripe was having
JWTToken, which is redundant.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When I looked over it, while KeyMode was only related to JWT right now, the NGINX OIDC Module lets you specify a local secret, as well as a remote URL. So this could eventually be re-used when we implement OIDC auth
That being said, I'll look back over it and see if other fields where
JWTwas removed could still keep it.