[ARXIVCE-2684] upstream develop to master #326
Merged
Conversation
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
…er buffer size is 4k, and the claim == token being bloated is not a good idea.
…rovide alternate logout URL. claims gets id_token property.
2 - user_claims.py - token needs more diet. Gave up on including access and id tokens in the secret part. Only encrypt the user's property part. Access token is only needed for talking to Keycloak, and payload is for us. 3. Creating tapir session needs transaction. It is adding the tapir session successfully.
I'm not understanding why Keycloak made account has no "aud" while using legacy auth has it. In any rate, I think we don't care where the accounts come from. Token pack/unpack was totally busted, and now fixed.
…toke for it. The payload size is a bit tight.
…_id doesn't change while email may.
User claim's user ID "may not be integer" in rare occasion so be defensive. Fix the refresh token handling.
…er than the user claims.
… This saves about 1k in size. validate_access_token may not be RSA key. This isn't implementing the other key types but needs some research that which key type is the smallest.
[WIP] OpenID connect client library - eyeing toward deploying keycloak
There is no `engine` in arxiv.db (any more?)
…atexml-db [ARXIVCE-2684] set the postgresql isolation level to READ UNCOMITTED
jonathanhyoung
approved these changes
Oct 16, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
PR brings #299 and #325 from develop to master