-
Notifications
You must be signed in to change notification settings - Fork 370
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
fix(app-check): support httpAgent
in JwksFetcher
#2689
Conversation
f94cf42
to
ae80c99
Compare
ae80c99
to
6b55c00
Compare
httpAgent
in JwksFetcher
httpAgent
in JwksFetcher
httpAgent
in JwksFetcher
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @hermanho for your contribution! LGTM!
For the CI build failed in 22.x, it is related to a bug in nodejs v22.7.0. It should be fixed in v22.8.0. I tested with v22.6.0 in local machine and no problem in emulator-based integration tests. |
Yep! Looks like they plan to release 22.8.0 next week. Let's wait and rerun the tests next week. If the Node release is delayed I can update the workflows to pin to 22.6.0 |
Fix: support httpAgent in JwksFetcher #2684