8000 Caching Public Keys for ID Token and Cookie Verification by hiranya911 · Pull Request #150 · firebase/firebase-admin-python · GitHub
[go: up one dir, main page]

Skip to content

Caching Public Keys for ID Token and Cookie Verifi 8000 cation #150

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 5 commits into from
Apr 5, 2018

Conversation

hiranya911
Copy link
Contributor

The google-auth library does not cache public certs. This causes all invocations of verify_id_token() and verify_session_cookie() to make an HTTP call. This PR implements certificate caching using the cachecontrol library.

Copy link
Contributor
@avishalom avishalom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hiranya911 hiranya911 assigned hiranya911 and unassigned avishalom Apr 5, 2018
@hiranya911 hiranya911 merged commit dbd8da5 into master Apr 5, 2018
@hiranya911 hiranya911 deleted the hkj-cache-control branch April 5, 2018 18:58
hiranya911 added a commit that referenced this pull request Apr 5, 2018
hiranya911 added a commit that referenced this pull request Apr 11, 2018
* Moved token generation/validation code to new helper module

* Basic session cookie support (without tests)

* Separated token generation and verification into two classes

* Added unit tests for session management

* Fixing a lint error

* Added integration tests

* Handling article in error messages

* Fixed a lint error

* Updated changelog

* Added snippets for auth session management

* Updated snippets

* Added some comments

* Merged with master; Updated CHANGELOG for #150

* Minor improvements to samples
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0