Skip to content

[5.1] Add | Cache TokenCredential objects to take advantage of token caching (#2380) #2776

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
Aug 16, 2024

Conversation

dauinsight
Copy link
Contributor

Backports: #2380

@dauinsight dauinsight added the Enhancement 💡 Issues that are feature requests for the drivers we maintain. label Aug 14, 2024
Copy link

codecov bot commented Aug 15, 2024

Codecov Report

Attention: Patch coverage is 82.10526% with 17 lines in your changes missing coverage. Please review.

Project coverage is 71.95%. Comparing base (6fe8e21) to head (dc6e465).
Report is 1 commits behind head on release/5.1.

Files Patch % Lines
...SqlClient/ActiveDirectoryAuthenticationProvider.cs 82.10% 17 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff               @@
##           release/5.1    #2776      +/-   ##
===============================================
+ Coverage        71.88%   71.95%   +0.07%     
===============================================
  Files              293      293              
  Lines            61574    61643      +69     
===============================================
+ Hits             44261    44355      +94     
+ Misses           17313    17288      -25     
Flag Coverage Δ
addons 92.38% <ø> (ø)
netcore 76.00% <82.10%> (+<0.01%) ⬆️
netfx 69.65% <82.10%> (+0.10%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@dauinsight dauinsight merged commit ca8746f into dotnet:release/5.1 Aug 16, 2024
132 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Enhancement 💡 Issues that are feature requests for the drivers we maintain.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants