Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

-keyalg RSA is needed or you'll get SSLHandshakeException: no cipher … #757

Merged
merged 2 commits into from
Aug 15, 2018

Conversation

StephanWald
Copy link
Contributor

This took me a short while to figure out - I just copy and pasted the keytool statement from the comment line and got SSLHandshakeException. If you add -keyalg RSA all works as described. I suggest adding this to the sample line so others who try this out will be on track immediately

@marci4
Copy link
Collaborator

marci4 commented Aug 15, 2018

Hello @StephanWald,

thank you for your pull request.

Just for the future, you do not need to close a pull request, just commit the requested changes to the specific branch and the pull request will get updated!

Best regards,
marci4

@marci4 marci4 merged commit 64b7574 into TooTallNate:master Aug 15, 2018
@marci4 marci4 added this to the Release 1.4.0 milestone Aug 15, 2018
@marci4 marci4 added the wss label Aug 15, 2018
@marci4 marci4 added the Bug label Oct 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants