Skip to content

fix: Add GraalVM configuration for the JDK's DNS Resolver.#2123

Merged
hessjcg merged 1 commit intomainfrom
graalvm-dns-fix
Mar 5, 2025
Merged

fix: Add GraalVM configuration for the JDK's DNS Resolver.#2123
hessjcg merged 1 commit intomainfrom
graalvm-dns-fix

Conversation

@hessjcg
Copy link
Copy Markdown
Collaborator

@hessjcg hessjcg commented Mar 5, 2025

This fixes the GraalVM distribution so that it can resolve DNS Names. GraalVM
needed the JDK's built-in DNS implementation to be explicitly configured, because
the DNS implementation classes are only referenced indirectly through reflection.

@hessjcg
Copy link
Copy Markdown
Collaborator Author

hessjcg commented Mar 5, 2025

This PR had a successful run of the GraalVM E2E tests

This fixes the GraalVM distribution so that it can resolve DNS Names.
@hessjcg hessjcg marked this pull request as ready for review March 5, 2025 15:05
@hessjcg hessjcg requested a review from a team as a code owner March 5, 2025 15:05
Copy link
Copy Markdown
Collaborator

@jackwotherspoon jackwotherspoon left a comment

Choose a reason for hiding this comment

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

LGTM ✅

@hessjcg hessjcg merged commit d276672 into main Mar 5, 2025
17 checks passed
@hessjcg hessjcg deleted the graalvm-dns-fix branch March 5, 2025 15:29
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