Remote JMX SSL bug fix and updated tests
This is a follow up to this pull request here: https://github.com/oracle/graal/pull/6250
It adds a reflection configuration that was missing, but needed for a JMX client to connect using SSL. The tests have been updated to connect with SSL and password authentication.
I've included dummy password, access, and SSL files required to configure the JMX properties needed for authentication and SSL.
Hi @fniephaus here's the fix I was mentioning and also the corresponding test updates.
Hi @fniephaus! I've added a README.md explaining the steps needed to create the supporting dummy SSL and password files. I've also rebased with master, fixed the conflicts, and added the fix from https://github.com/oracle/graal/pull/11568 so that the tests can pass.
GraalVM Gate // compiler style,fullbuild,test JDKlatest is failing on multiple other recent PRs and seems to be related to the compiler, not this PR.
Please have another look at this PR when you have time.
I went over the PR and added a commit (see https://github.com/oracle/graal/pull/11599).
An internal reviewer mentioned we need to get rid of the binary files. Is there any way you could build on https://github.com/oracle/graal/pull/11599/commits/19c02bdfc26d83fcb426a03feddb5b05f91c907a and make is so that the keytool is invoked in the temporary directory?
I went over the PR and added a commit (see #11599).
An internal reviewer mentioned we need to get rid of the binary files. Is there any way you could build on 19c02bd and make is so that the keytool is invoked in the temporary directory?
Hi @fniephaus! Thank you for doing that. I've cherry-picked your commit and added a new commit on top of it here. The new commit uses keytool programmatically and creates the ssl files in the temporary directory.
Thanks for taking care of this, @roberttoyonaga. I have refactored the test some more and now, all files are inlined into the test file and on-the-fly created in the temp directory.
if you could reset your branch to this commit and squash all commits, I think this is ready to be integrated.
I have refactored the test some more and now, all files are inlined into the test file and on-the-fly created in the temp directory. if you could reset your branch to this commit and squash all commits, I think this is ready to be integrated.
Great, thank you @fniephaus. I've reset and squashed the commits.