Skip to content

Conversation

@tvernum
Copy link
Contributor

@tvernum tvernum commented Jul 3, 2025

Backports the following commits to 9.1:

In Java 14 the `MessageDigest` specification was changed so that the
"MD5" hash function is no longer required. It is permissible for a JRE
to ship without support for MD5 hashes.

This commit modifies the ES|QL MD5 hash function implementation so
that if the specified `MessageDigest` object is not available on startup,
then the error is non-fatal, and the node will still boot successfully.

If an ES|QL query attempts to make use of md5, and it is unavailable,
then the query will fail with an ES|QL verification exception

Resolves: elastic#129689
@tvernum tvernum added :Analytics/ES|QL AKA ESQL >bug auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) labels Jul 3, 2025
@tvernum tvernum requested a review from a team as a code owner July 3, 2025 08:53
@tvernum tvernum added >bug backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Analytics/ES|QL AKA ESQL labels Jul 3, 2025
@elasticsearchmachine elasticsearchmachine merged commit 2a8e3f7 into elastic:9.1 Jul 3, 2025
32 checks passed
@tvernum tvernum deleted the backport/9.1/pr-130158 branch July 3, 2025 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >bug Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v9.1.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants