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 2.9] fix password argument in SecureRestClientBuilder constructor for SecureWorkflowRestApiIT #1042

Merged
merged 1 commit into from
Jul 19, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport e4fae0e from #1041

…reWorkflowRestApiIT (#1041)

* fix password argument in SecureRestClientBuilder constructor for SecureWorkflowRestApiIT

Signed-off-by: Surya Sashank Nistala <snistala@amazon.com>

* fix tests to account for AUDIT alerts

Signed-off-by: Surya Sashank Nistala <snistala@amazon.com>

---------

Signed-off-by: Surya Sashank Nistala <snistala@amazon.com>
(cherry picked from commit e4fae0e)
@codecov
Copy link

codecov bot commented Jul 17, 2023

Codecov Report

Merging #1042 (f97c668) into 2.9 (aefb268) will decrease coverage by 0.26%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                2.9    #1042      +/-   ##
============================================
- Coverage     72.85%   72.59%   -0.26%     
  Complexity      119      119              
============================================
  Files           160      160              
  Lines         10448    10448              
  Branches       1572     1572              
============================================
- Hits           7612     7585      -27     
- Misses         1944     1972      +28     
+ Partials        892      891       -1     

see 8 files with indirect coverage changes

@eirsep eirsep added the v2.9.0 v2.9.0 label Jul 19, 2023
@eirsep eirsep merged commit e55c0c9 into 2.9 Jul 19, 2023
@github-actions github-actions bot deleted the backport/backport-1041-to-2.9 branch July 19, 2023 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v2.9.0 v2.9.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants