[16.0] mig auth_jwt and auth_jwt_demo#519
Merged
OCA-git-bot merged 34 commits intoOCA:16.0from Jun 7, 2023
Merged
Conversation
Member
Author
|
/ocabot migration auth_jwt |
Member
Author
|
/ocabot migration auth_jwt_demo |
f9a442a to
27e9f13
Compare
Because it allows validating with a list of audiences.
Make it work with pyjwt.
This method is useful for public endpoints that need to work for anonymous user, but can be enhanced when an authenticated user is know. A typical use case is a "add to cart" enpoint that can work for anonymous users, but can be enhanced by binding the cart to a known customer when the authenticated user is known.
95ebfc8 to
66699ab
Compare
The unit tests were broken for non-functional reasons (interaction with the mock) and is easier to implement as integration test.
66699ab to
35f4fd6
Compare
hparfr
approved these changes
Jun 7, 2023
Member
Author
|
/ocabot merge nobump |
Contributor
|
This PR looks fantastic, let's merge it! |
Contributor
|
Congratulations, your PR was merged at 474ee20. Thanks a lot for contributing to OCA. ❤️ |
OCA-git-bot
pushed a commit
that referenced
this pull request
Jul 2, 2023
alan196
pushed a commit
to Jarsa/server-auth
that referenced
this pull request
Jul 3, 2023
remihb
pushed a commit
to osiell/server-auth
that referenced
this pull request
Sep 6, 2023
amkarthik
pushed a commit
to sodexis/server-auth
that referenced
this pull request
Nov 21, 2023
moncefarajdal
pushed a commit
to moncefarajdal/server-auth
that referenced
this pull request
Oct 15, 2024
SiesslPhillip
pushed a commit
to grueneerde/OCA-server-auth
that referenced
this pull request
Nov 20, 2024
Syncing from upstream OCA/server-auth (14.0)
SodexisTeam
pushed a commit
to sodexis/server-auth
that referenced
this pull request
Dec 3, 2024
BT-dlagin
pushed a commit
to BT-dlagin/server-auth
that referenced
this pull request
Dec 20, 2024
adhoc-cicd-bot
pushed a commit
to adhoc-cicd/oca-server-auth
that referenced
this pull request
Jan 13, 2025
adhoc-cicd-bot
pushed a commit
to adhoc-cicd/oca-server-auth
that referenced
this pull request
Jan 18, 2025
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
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.
Migrating both modules in the same PR since we need the demo module to test the first.