Skip to content

KNOX-3315: Adds BCFKS as an option cert export#1221

Merged
hanicz merged 2 commits into
apache:masterfrom
hanicz:KNOX-3315
May 6, 2026
Merged

KNOX-3315: Adds BCFKS as an option cert export#1221
hanicz merged 2 commits into
apache:masterfrom
hanicz:KNOX-3315

Conversation

@hanicz
Copy link
Copy Markdown
Contributor

@hanicz hanicz commented May 6, 2026

KNOX-3315 - Update publicCert endpoint to be able to generate BCFKS as well

What changes were proposed in this pull request?

Adds the ability for the publicCert endpoint to be able to generate BCFKS as well. Added the same for the KnoxCLI export-cert command.

How was this patch tested?

Unit tests, tested on FIPS cluster where the necessary BC FIPS provider for BCFKS is there.

/usr/java/default/bin/keytool -list -keystore gateway-client-trust.bcfks -v -storetype BCFKS -providerclass com.safelogic.cryptocomply.jcajce.provider.CryptoComplyFipsProvider -providerpath com-safelogic-cryptocomply-fips-core.jar -storepass XXX

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

Test Results

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

Results for commit ffa0211.

♻️ This comment has been updated with latest results.

Comment thread gateway-server/src/main/java/org/apache/knox/gateway/util/KnoxCLI.java Outdated
@smolnar82
Copy link
Copy Markdown
Contributor

I also created a follow-up JIRA to have this option on the Knox Home page too (see KNOX-3316).

Copy link
Copy Markdown
Contributor

@moresandeep moresandeep left a comment

Choose a reason for hiding this comment

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

Looks good!

@hanicz hanicz merged commit df17516 into apache:master May 6, 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.

3 participants