-
Notifications
You must be signed in to change notification settings - Fork 191
RUST-1662: Correctly handle Client and Connection caching, add Speculative Auth #1040
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
Merged
Merged
Changes from all commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
db62e00
RUST-1662: Initial caching checkpoint
pmeredit 5738c04
RUST-1662: Finish non-speculative initial authentication
pmeredit e0ba578
RUST-1662: Refactor out two_step_auth
pmeredit 3d73dd0
RUST-1662: Use constants, use u32 instead of i32 for things that cann…
pmeredit 014ae5d
RUST-1662: Add speculative command building
pmeredit 43f934a
RUST-1662: Update validation and and comments
pmeredit cf0ad9f
RUST-1662: Make oidc test patchable
pmeredit 6cf4d20
RUST-1662: Add human test
pmeredit db511b2
RUST-1662: Update error
pmeredit cff6abc
Update src/client/auth/oidc.rs
pmeredit 80f8b06
RUST-1662: Fix problems with spec auth
pmeredit b3cf7f5
RUST-1662: Minor cleanup
pmeredit fe22a70
RUST-1662: Why is validate failing in the tests?
pmeredit a58357e
RUST-1662: use multithreaded?
pmeredit 376b2f3
RUST-1662: Ok, looks like we need to propagate async. The change was …
pmeredit b958989
RUST-1662: See if the validate failure was just caused by a misuse of…
pmeredit faee44e
RUST-1662: Add callback so validate doesn't fail
pmeredit df0fd63
RUST-1662: Do better, don't add empty callback, also don't require ca…
pmeredit 900c9e6
Update src/client/auth/oidc.rs
pmeredit 0c909bd
RUST-1662: Refactor and ensure that idpserver info and refresh token …
pmeredit a8c3588
RUST-1662: Change to use token_gen_id like the python driver
pmeredit f5c20e2
RUST-1662: Use consistent naming between the two cache functions
pmeredit 6a527a0
RUST-1662: rustfmt?
pmeredit e80dc57
RUST-1662: Use std::sync::RwLock for Client Cache
pmeredit a7099b4
RUST-1662: Connection Cache can also be sync since it's not public
pmeredit ce319ad
RUST-1662: Looks like the rustfmt problem is with stable vs nightly
pmeredit b3e8be5
RUST-1662: Fix unnecessary async, reuse method
pmeredit 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 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -295,9 +295,9 @@ buildvariants: | |
|
|
||
| - name: oidc | ||
| display_name: OIDC | ||
| patchable: false | ||
| patchable: true | ||
| run_on: | ||
| - rhel87-small | ||
| - ubuntu2204-small | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This was just so the oidc tests would start immediately |
||
| expansions: | ||
| AUTH: auth | ||
| SSL: ssl | ||
|
|
||
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
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.
This will be removed before merge, again :)