-
Notifications
You must be signed in to change notification settings - Fork 358
Don't send a auth header when fetching the versions during a token refresh #5822
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
Conversation
pixlwave
left a comment
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.
Not that my approval counts for much, but thank you! 🙏
c5a8f87 to
e5d7434
Compare
Hywan
left a comment
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.
Looks good to me, thanks! It's clear and tested, all good.
e5d7434 to
23d2b5b
Compare
CodSpeed Performance ReportMerging #5822 will not alter performanceComparing Summary
|
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #5822 +/- ##
==========================================
- Coverage 88.52% 88.51% -0.01%
==========================================
Files 361 361
Lines 101331 101367 +36
Branches 101331 101367 +36
==========================================
+ Hits 89702 89730 +28
- Misses 7426 7434 +8
Partials 4203 4203 ☔ View full report in Codecov by Sentry. |
This closes #5816.