Skip to content

KNOX-3311: Fix X509CertificateUtil.fetchPublicCertsFromServer issue w…#1217

Merged
hanicz merged 2 commits intoapache:masterfrom
hanicz:KNOX-3311
Apr 29, 2026
Merged

KNOX-3311: Fix X509CertificateUtil.fetchPublicCertsFromServer issue w…#1217
hanicz merged 2 commits intoapache:masterfrom
hanicz:KNOX-3311

Conversation

@hanicz
Copy link
Copy Markdown
Contributor

@hanicz hanicz commented Apr 29, 2026

…here default truststore can't be loaded due to invalid/missing password

KNOX-3311 - Fix X509CertificateUtil.fetchPublicCertsFromServer issue where default truststore can't be loaded due to invalid/missing password

What changes were proposed in this pull request?

Fix X509CertificateUtil.fetchPublicCertsFromServer issue where default truststore can't be loaded due to invalid/missing password

How was this patch tested?

Tested on FIPS cluster with BCFKS default truststore
Tested locally where the password is not set

Integration Tests

N/A

UI changes

N/A

…here default truststore can't be loaded due to invalid/missing password
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 29, 2026

Test Results

21 tests   21 ✅  1s ⏱️
 1 suites   0 💤
 1 files     0 ❌

Results for commit fb2c342.

♻️ This comment has been updated with latest results.

@hanicz hanicz marked this pull request as ready for review April 29, 2026 10:30
@hanicz hanicz requested a review from smolnar82 April 29, 2026 10:30
Copy link
Copy Markdown
Contributor

@smolnar82 smolnar82 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hanicz hanicz merged commit 4f94d95 into apache:master Apr 29, 2026
3 checks passed
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.

2 participants