firebase / firebase-admin-go

Firebase Admin Go SDK
Apache License 2.0
1.12k stars 239 forks source link

fix(auth): fail gracefully when no cache headers are available #623

Closed lahirumaramba closed 1 month ago

lahirumaramba commented 1 month ago

Resolves: #621

Avoid throwing errors when parsing cache control headers for public keys and set the cache duration to 0 (equivalent to no cache) when the cache headers cannot be parsed.

lahirumaramba commented 1 month ago

LGTM. Thank you so much for the quick response Lahiru!

qq - r we able to test this against a real response?

Yeah, the integrations tests are all green!