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.x] Bump BouncyCastle from jdk15on to jdk15to18 (#2901) #2917

Merged

Conversation

willyborankin
Copy link
Collaborator

Backport 9a72355 from #2901

@codecov
Copy link

codecov bot commented Jun 29, 2023

Codecov Report

Merging #2917 (668b6f1) into 2.x (101f58c) will decrease coverage by 0.08%.
The diff coverage is 52.94%.

@@             Coverage Diff              @@
##                2.x    #2917      +/-   ##
============================================
- Coverage     62.13%   62.05%   -0.08%     
+ Complexity     3374     3368       -6     
============================================
  Files           264      264              
  Lines         19441    19440       -1     
  Branches       3315     3314       -1     
============================================
- Hits          12080    12064      -16     
- Misses         5767     5776       +9     
- Partials       1594     1600       +6     
Impacted Files Coverage Δ
...ensearch/security/ssl/DefaultSecurityKeyStore.java 67.26% <52.94%> (-0.27%) ⬇️

... and 6 files with indirect coverage changes

@RyanL1997
Copy link
Collaborator

Hey @willyborankin, thank you for taking care of the backport. The style check is failing due to this error:

Error: eckstyle] [ERROR] /home/runner/work/security/security/src/main/java/org/opensearch/security/ssl/DefaultSecurityKeyStore.java:20:8: Unused import - java.io.ByteArrayInputStream. [UnusedImports]

@willyborankin
Copy link
Collaborator Author

willyborankin commented Jun 29, 2023

Hey @willyborankin, thank you for taking care of the backport. The style check is failing due to this error:

Error: eckstyle] [ERROR] /home/runner/work/security/security/src/main/java/org/opensearch/security/ssl/DefaultSecurityKeyStore.java:20:8: Unused import - java.io.ByteArrayInputStream. [UnusedImports]

Oops will fix

@willyborankin willyborankin force-pushed the backport/backport-2901-to-2.x branch from 51e86b1 to b7bdbde Compare June 29, 2023 17:05
…h-project#2901)

jdk15to18 contains fix for
 - CVE-2023-33201 - Medium
   Severity Vulnerability

Signed-off-by: Andrey Pleskach <[email protected]>
(cherry picked from commit 9a72355)
Signed-off-by: Andrey Pleskach <[email protected]>
@willyborankin willyborankin force-pushed the backport/backport-2901-to-2.x branch from b7bdbde to 668b6f1 Compare June 29, 2023 19:32
@willyborankin
Copy link
Collaborator Author

Hey @willyborankin, thank you for taking care of the backport. The style check is failing due to this error:

Error: eckstyle] [ERROR] /home/runner/work/security/security/src/main/java/org/opensearch/security/ssl/DefaultSecurityKeyStore.java:20:8: Unused import - java.io.ByteArrayInputStream. [UnusedImports]

Oops will fix

fixed

@cwperks cwperks merged commit 010e444 into opensearch-project:2.x Jun 30, 2023
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.

4 participants