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] Set default value for key/trust store type as constant for JDK PKCS setup #5003

Merged
merged 1 commit into from
Jan 2, 2025

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport e83db44 from #5000.

…etup (#5000)

Signed-off-by: Andrey Pleskach <[email protected]>
(cherry picked from commit e83db44)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Copy link

codecov bot commented Jan 2, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.91%. Comparing base (830f9e1) to head (e1c1feb).
Report is 1 commits behind head on 2.x.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##              2.x    #5003      +/-   ##
==========================================
- Coverage   63.93%   63.91%   -0.03%     
==========================================
  Files         330      330              
  Lines       23185    23185              
  Branches     3755     3755              
==========================================
- Hits        14824    14819       -5     
- Misses       6528     6532       +4     
- Partials     1833     1834       +1     
Files with missing lines Coverage Δ
...rch/security/ssl/config/SslCertificatesLoader.java 88.67% <100.00%> (ø)
...ensearch/security/ssl/util/SSLConfigConstants.java 79.16% <ø> (ø)

... and 3 files with indirect coverage changes

@cwperks cwperks merged commit 18cea11 into 2.x Jan 2, 2025
91 checks passed
@cwperks cwperks deleted the backport/backport-5000-to-2.x branch January 2, 2025 14:48
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.

1 participant