Skip to content
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

Backport- Fixed race condition in AWSV4SignerAuth & AWSV4SignerAsyncAuth when using refreshable credentials #473

Merged
merged 1 commit into from
Aug 15, 2023

Conversation

saimedhi
Copy link
Collaborator

Description

Backport #470 to 2.3. Fixed race condition in AWSV4SignerAuth & AWSV4SignerAsyncAuth when using refreshable credentials

Issues Resolved

Related to #472

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@codecov
Copy link

codecov bot commented Aug 15, 2023

Codecov Report

Merging #473 (6da9190) into 2.3 (f108677) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##              2.3     #473   +/-   ##
=======================================
  Coverage   70.91%   70.92%           
=======================================
  Files          81       81           
  Lines        7730     7732    +2     
=======================================
+ Hits         5482     5484    +2     
  Misses       2248     2248           
Files Changed Coverage Δ
opensearchpy/helpers/asyncsigner.py 95.83% <100.00%> (+0.18%) ⬆️
opensearchpy/helpers/signer.py 94.73% <100.00%> (+0.14%) ⬆️

@saimedhi saimedhi merged commit b20b027 into 2.3 Aug 15, 2023
@github-actions github-actions bot deleted the backport/backport-470-to-2.3 branch August 15, 2023 17:19
@saimedhi saimedhi restored the backport/backport-470-to-2.3 branch August 15, 2023 17:31
saimedhi added a commit that referenced this pull request Aug 15, 2023
…h when using refreshable credentials (#473)"

This reverts commit b20b027.
@saimedhi
Copy link
Collaborator Author

Sorry, I missed DCO. I hope it won't create any problem. Can I release this patch version?

@dblock
Copy link
Member

dblock commented Aug 15, 2023

I force-pushed a DCO fix to 2.3.

Let's go a little slower and not merge things on red.

dblock pushed a commit that referenced this pull request Aug 15, 2023
…sing refreshable credentials (#473)

Co-authored-by: Logan Attwood <lattwood@users.noreply.github.com>
Signed-off-by: dblock <dblock@amazon.com>
@saimedhi saimedhi deleted the backport/backport-470-to-2.3 branch September 6, 2023 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants