Skip to content

Support SecureRandom strong algorithms in FIPS mode - #446

Open
Mohit-Rajbhar100698 wants to merge 1 commit into
ibmruntimes:openj9from
Mohit-Rajbhar100698:backport/21/fips-secureRandom-strongAlgo
Open

Support SecureRandom strong algorithms in FIPS mode#446
Mohit-Rajbhar100698 wants to merge 1 commit into
ibmruntimes:openj9from
Mohit-Rajbhar100698:backport/21/fips-secureRandom-strongAlgo

Conversation

@Mohit-Rajbhar100698

@Mohit-Rajbhar100698 Mohit-Rajbhar100698 commented Jul 2, 2026

Copy link
Copy Markdown

This change overrides the default securerandom.strongAlgorithms
configuration used by SecureRandom.getInstanceStrong() when it
runs in FIPS mode.

This PR also includes two additional backports. One removes the
provider-specific condition from the SecureRandom strongAlgorithms
test. The other handles the Windows-specific SecureRandom
strongAlgorithms defaults used by the test.

Back-ported from: #1187 #1225 #1232

Signed-off-by: Mohit Rajbhar mohit.rajbhar@ibm.com

@pshipton
pshipton marked this pull request as draft July 7, 2026 13:10
@Mohit-Rajbhar100698
Mohit-Rajbhar100698 marked this pull request as ready for review July 15, 2026 13:00
@Mohit-Rajbhar100698
Mohit-Rajbhar100698 force-pushed the backport/21/fips-secureRandom-strongAlgo branch from 203ebbe to 4c4013b Compare July 15, 2026 13:03
@Mohit-Rajbhar100698

Copy link
Copy Markdown
Author

Hi @keithc-ca
could you review this PR whenever you have a chance?

@keithc-ca
keithc-ca self-requested a review July 16, 2026 20:00
@pshipton
pshipton marked this pull request as draft July 16, 2026 20:31
Comment thread closed/test/jdk/openj9/internal/security/TestProperties.java Outdated
@Mohit-Rajbhar100698

Copy link
Copy Markdown
Author

Hi @keithc-ca , is there anything left for me to address in this PR?

@keithc-ca

Copy link
Copy Markdown
Member

Yes, I was expecting back-ports of ibmruntimes/openj9-openjdk-jdk#1225 and ibmruntimes/openj9-openjdk-jdk#1232 to be proposed and merged before this.

@Mohit-Rajbhar100698

Copy link
Copy Markdown
Author

Ok when this current PR merges. Then Only I Will be able to raise PR for #1225 and finally #1232 backports

As the next two PR contains changes on the top of this feature
Right ?

@keithc-ca

Copy link
Copy Markdown
Member

the next two PR contains changes on the top of this feature

Sorry, you're right. I thought the other changes were more independent of this. I'll have another look at this with that in mind.

@keithc-ca

Copy link
Copy Markdown
Member

With my corrected understanding of the relationships between ibmruntimes/openj9-openjdk-jdk#1187, ibmruntimes/openj9-openjdk-jdk#1225 and ibmruntimes/openj9-openjdk-jdk#1232, I think it will be simpler to manage if this is updated so it back-ports all three changes (and makes clear that it back-ports three changes, not just one).

@Mohit-Rajbhar100698

Copy link
Copy Markdown
Author

With my corrected understanding of the relationships between ibmruntimes/openj9-openjdk-jdk#1187, ibmruntimes/openj9-openjdk-jdk#1225 and ibmruntimes/openj9-openjdk-jdk#1232, I think it will be simpler to manage if this is updated so it back-ports all three changes (and makes clear that it back-ports three changes, not just one).

Just to confirm, are you suggesting that we should have a single backport PR that includes all the changes from #1187, #1225, and #1232, instead of backporting them as three separate PRs?

In PR Description we can write
Back-ported from:

  • #1187
  • #1225
  • #1232

?

@keithc-ca

Copy link
Copy Markdown
Member

a single backport PR

Yes.

This change overrides the default securerandom.strongAlgorithms
configuration used by SecureRandom.getInstanceStrong() when it
runs in FIPS mode.

This PR also includes two additional backports. One removes the
provider-specific condition from the SecureRandom strongAlgorithms
test(#1225). The other handles the Windows-specific SecureRandom
strongAlgorithms defaults used by the test(#1232).

Signed-off-by: Mohit Rajbhar <mohit.rajbhar@ibm.com>
@Mohit-Rajbhar100698
Mohit-Rajbhar100698 force-pushed the backport/21/fips-secureRandom-strongAlgo branch from 5b82673 to 7b8aba0 Compare July 31, 2026 20:23
@Mohit-Rajbhar100698

Copy link
Copy Markdown
Author

Hi @keithc-ca
Updated this PR with all backports. Can u review once.

@keithc-ca
keithc-ca marked this pull request as ready for review July 31, 2026 20:55
@keithc-ca

Copy link
Copy Markdown
Member

Jenkins test sanity alinux jdk21

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