Skip to content

Conversation

DaveCTurner
Copy link
Contributor

These tests only don't work in a FIPS JVM because they use a secret key
that is unacceptably short. This commit replaces the relevant uses of
randomIdentifier with randomSecretKey so they work whether in FIPS
mode or not.

Backport of #126109 to 8.x

These tests only don't work in a FIPS JVM because they use a secret key
that is unacceptably short. This commit replaces the relevant uses of
`randomIdentifier` with `randomSecretKey` so they work whether in FIPS
mode or not.

Backport of elastic#126109 to `8.x`
@DaveCTurner DaveCTurner added >test Issues or PRs that are addressing/adding tests :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs backport :Security/FIPS Running ES in FIPS 140-2 mode auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) v8.19.0 labels Apr 4, 2025
@DaveCTurner DaveCTurner added the Team:Security Meta label for security team label Apr 4, 2025
@elasticsearchmachine elasticsearchmachine merged commit b121ee2 into elastic:8.x Apr 4, 2025
20 checks passed
@DaveCTurner DaveCTurner deleted the 2025/04/04/backport-126109-S3SearchableSnapshotsCredentialsReloadIT-fips-8x branch April 4, 2025 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs :Security/FIPS Running ES in FIPS 140-2 mode Team:Security Meta label for security team >test Issues or PRs that are addressing/adding tests v8.19.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants