Skip to content

Conversation

@github-actions
Copy link
Contributor

Proposed changes

The request headers and request body are not required for the jwks uri subrequest.

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto main
  • I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

@github-actions github-actions bot requested a review from a team as a code owner August 12, 2025 11:21
@github-actions github-actions bot added bug An issue reporting a potential bug needs cherry pick Cherry pick this PR into a release branch labels Aug 12, 2025
AlexFenlon
AlexFenlon previously approved these changes Aug 12, 2025
pdabelf5
pdabelf5 previously approved these changes Aug 12, 2025
@pdabelf5 pdabelf5 closed this Aug 12, 2025
@pdabelf5 pdabelf5 reopened this Aug 12, 2025
@github-actions github-actions bot removed the bug An issue reporting a potential bug label Aug 12, 2025
@pdabelf5 pdabelf5 dismissed stale reviews from AlexFenlon and themself via d52a94a August 12, 2025 13:00
@codecov
Copy link

codecov bot commented Aug 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (release-5.1@d05b3de). Learn more about missing BASE report.

Additional details and impacted files
@@              Coverage Diff               @@
##             release-5.1    #8125   +/-   ##
==============================================
  Coverage               ?   52.98%           
==============================================
  Files                  ?       90           
  Lines                  ?    21746           
  Branches               ?        0           
==============================================
  Hits                   ?    11523           
  Misses                 ?     9746           
  Partials               ?      477           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pdabelf5 pdabelf5 merged commit ad2cf79 into release-5.1 Aug 12, 2025
72 checks passed
@pdabelf5 pdabelf5 deleted the cherry-pick-release-5.1-4ef38ae5107398f4dda1a227596195527802bb0a branch August 12, 2025 14:09
@pdabelf5 pdabelf5 changed the title [cherry-pick] Don't send request headers & body to jwks uri Don't send request headers & body to jwks uri Aug 12, 2025
@pdabelf5 pdabelf5 added bug An issue reporting a potential bug and removed needs cherry pick Cherry pick this PR into a release branch labels Aug 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug An issue reporting a potential bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants