-
Notifications
You must be signed in to change notification settings - Fork 1.2k
feat(spanner): Support dynamic certificate and key rotation in Spanner Omni #14433
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
Closed
Closed
Changes from all commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
0fb7c7e
feat(spanner): Support Dynamic Certificate/Key rotation in Spanner Omni
sagnghos 8473bb7
fix(spanner): Throttle key/trust manager file stat checks and use nat…
sagnghos c5ec704
fix(spanner): Detect PKCS#1 keys with actionable error and clean up t…
sagnghos a077de2
Address review comments: avoid capturing builder in SpannerOptions la…
sagnghos 959f887
Address review comments: simplify reloadMaterial and suppress excepti…
sagnghos 7fcbedf
docs(spanner): Add Javadocs for ConnectionOptions builder and Dynamic…
sagnghos 6993ccf
fix(spanner): Validate private key matches certificate and optimize f…
sagnghos ab27a98
fix(spanner): Address review comments: versioned alias map, binary DE…
sagnghos f78415f
fix(spanner): Non-blocking Netty handshakes via background scheduled …
sagnghos 53f59f9
fix(spanner): On-demand rate-limited checking on TLS handshake, remov…
sagnghos c1f0f39
fix(spanner): Use monotonic System.nanoTime() and ReentrantLock for t…
sagnghos c031825
Store certificate and key paths in SpannerOptions and Builder
sagnghos 0b61f6e
Validate client certificate and key are both provided in prepareBuilder
sagnghos 8e80a78
Use Strings.isNullOrEmpty for certificate and key validation
sagnghos 0789fb9
Use lock.tryLock() in DynamicKeyManager and DynamicTrustManager to pr…
sagnghos c3ddcc7
Perform file attribute checks only after acquiring lock in DynamicKey…
sagnghos 1886649
Use removeIf on materials.keySet() in DynamicKeyManager
sagnghos 4da5bf9
Optimize parsePrivateKey and extractPemContent in DynamicKeyManager
sagnghos 27d701f
Safely evict key material aliases in DynamicKeyManager
sagnghos cacf044
Safely handle null issuers in DynamicTrustManager and speed up tests …
sagnghos a376228
Use standard JKS keystore in DynamicTrustManager
sagnghos 8d9c737
Use KeyStore.getDefaultType() and clone accepted issuers in DynamicTr…
sagnghos 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
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
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
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.
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.
Uh oh!
There was an error while loading. Please reload this page.