stop caching guest jwks response#1615
Conversation
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
WalkthroughThe guest JWKS endpoint caching policy has been changed to disable all HTTP caching. The Cache-Control header was updated from "public, max-age=3600" to "no-store" in the route configuration. Corresponding test assertions and code comments were updated to reflect this revised behavior, ensuring the JWKS data is not cached by edge caches or clients. 📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Why:
data:URI, but rejects it when fetching the remote JWKS URL.Cache-Control: no-store) so remote verifiers always fetch fresh key material.Files: