-
Notifications
You must be signed in to change notification settings - Fork 379
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
Clear token cache when hashing is disabled #2698
base: master
Are you sure you want to change the base?
Clear token cache when hashing is disabled #2698
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2698 +/- ##
============================================
- Coverage 56.30% 55.72% -0.58%
- Complexity 8595 8807 +212
============================================
Files 654 654
Lines 49393 50839 +1446
Branches 9950 10470 +520
============================================
+ Hits 27811 28331 +520
- Misses 17621 18474 +853
- Partials 3961 4034 +73
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
ed1de7a
to
5097dee
Compare
PR builder started |
PR builder completed |
PR builder started |
PR builder completed |
@HasiniSama let's analyze integration test failures and update the code/ tests accordingly. Converting the PR to a draft until the tests are fixed |
Hi @HasiniSama, Please take one of the following actions:
Your prompt attention to this matter is greatly appreciated. Thank you for your understanding and collaboration! 🙏 |
Proposed changes in this pull request
$subject
This fix will hash the access token before sending it for cache clearance when hashing is disabled.