-
Notifications
You must be signed in to change notification settings - Fork 77
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
refactor(jans-cedarling): make all tokens optional when calling Cedarling::authorize #10436
Merged
Merged
Changes from 21 commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
cdcb32d
refactor(jans-cedarling): make process_tokens params optional
rmarinn c1821f1
feat(jans-cedarling): implement processing single tokens
rmarinn 0a9c4bc
refactor(jans-cedarling): replace jwt validation implementation
rmarinn 7cacde6
feat(jans-cedarling): make tokens optional when calling authorize
rmarinn 9e44ddd
refactor(jans-cedarling): make creation of workload entity optional
rmarinn e7e8fea
fix(jans-cedarling): user and role entity creation
rmarinn 3b9d727
refactor(jans-cedarling): make creation of user entity optional
rmarinn 9b5eeba
refactor(jans-cedarling): use if-else instead of match for bool
rmarinn c26ce92
refactor(jans-cedarling): improve functionality for decoded JWTs
rmarinn 3d03121
chore(jans-cedarling): removed unused functions for TrustedIssuer
rmarinn 03fbe87
refactor(jans-cedarling): improve create_workload_entity func impleme…
rmarinn d547371
refactor(jans-cedarling): improve create_user_entity func implemenation
rmarinn d37fd2b
chore(jans-cedarling): resolve clippy issues
rmarinn 1a26b7a
refactor(jans-cedarling): error check when loading bootstrap config
rmarinn 7b3769f
Merge branch 'main' into jans-cedarling-10409
rmarinn 94c740b
refactor(jans-cedarling): add new fields to TokenEntityMetadata
rmarinn eda65f3
refactor(jans-cedarling): remove some cloning when logging authz
rmarinn 9b2e667
test(jans-cedarling): add unit tests for entity creation
rmarinn 0ac1105
refactor(jans-cedarling): improve error handling for entity creation
rmarinn 5ccf4ec
Merge branch 'main' into jans-cedarling-10408
rmarinn aa9c071
test(jans-cedarling): update python tests expected error
rmarinn 33bdfbf
feat(jans-cedarling): make tokens optional in the python binding
rmarinn e170008
chore(jans-cedarling): change error message to start with a lowercase…
rmarinn 2051f1c
chore(jans-cedarling): flattened `use` statements for readability
rmarinn 9bbf437
chore(jans-cedarling): remove unused JwtProcessingError variant
rmarinn 4a16d27
refactor(jans-cedarling): simplify iterator creation
rmarinn 818bda4
refactor(jans-cedarling): make workload entity creation optional
rmarinn 8a7ea74
refactor(jans-cedarling): make user entity creation optional
rmarinn 12b5386
chore(jans-cedarling): remove outdated comment
rmarinn 18ed767
refactor(jans-cedarling): convert Token into a struct from an enum
rmarinn d947320
refactor(jans-cedarling): unify token entity creation logic
rmarinn a38f03f
refactor(jans-cedarling): eliminate access_token from user entity cre…
rmarinn 6dd4ecc
fix(jans-cedarling): field name conflict for serializing/deserializing
rmarinn 3301ad9
chore(jans-cedarling): remove unused commented code
rmarinn ddd9c27
refactor(jans-cedarling): token priority when creating userinfo entity
rmarinn ca4d1eb
Merge branch 'main' into jans-cedarling-10408
rmarinn 028b73c
chore(jans-cedarling): resolve clippy issues
rmarinn 17fd13e
test(jans-cedarling): update expected error in python bindings test
rmarinn ac7f80c
refactor(jans-cedarling): implement rename for logging info fields
rmarinn 36525fa
chore(jans-cedarling): run cargo fmt and add missing license headers
rmarinn 05440f9
chore(jans-cedarling): rename the Vec of role from `role` to `roles`
rmarinn 87b399f
chore(jans-cedarling): remove unused function
rmarinn 220b39a
refactor(jans-cedarling): rename UserAuthorizeInfo fields
rmarinn f0030ab
chore(jans-cedarling): run cargo fmt
rmarinn a4cbbda
chore(jans-cedarling): refactor to avoid unnecessary cloning
olehbozhok 588198d
Merge branch 'main' into jans-cedarling-10408
rmarinn 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
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.
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.
Do you assume that using python binding we always have all tokens?
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.
fixed in 33bdfbf