-
Notifications
You must be signed in to change notification settings - Fork 359
feat(appcheck): Verify one-time tokens for replay protection #976
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
+189
−14
Merged
Changes from all commits
Commits
Show all changes
15 commits
Select commit
Hold shift + click to select a range
3e328ce
add one time token verification support
yvonnep165 02f5f74
add unit tests
yvonnep165 bd14d81
fix lint error
yvonnep165 df4fcf2
change to use _utils.handle_platform_error_from_requests
yvonnep165 96c7083
change the function description
yvonnep165 ccca469
change docstring to state limited-use tokens
yvonnep165 86d356d
add validation for consume variable
yvonnep165 3c00741
Merge branch 'main' into yp-verify-one-time-token
yvonnep165 4e2f108
add check_boolean to _Validators and add unit tests to check non-bool…
yvonnep165 40e96d2
raise `exceptions.UnknownError` if body is not a dict
yvonnep165 7a95f91
move the replay verification to a separate helper method and add unit…
yvonnep165 5ce0edb
add test_verify_token_with_consume_false_makes_no_http_call unit test
yvonnep165 0957eed
replace the broad exceptions.FirebaseError assertion with more specif…
yvonnep165 d43471f
Refactor token verification tests to use mock_jwt fixture
yvonnep165 f500ef4
Document ValueError when consume is not a boolean in verify_token doc…
yvonnep165 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
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.