Skip to content

Add ECDHE-RSA-AES128-GCM-SHA256 support #185

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

Conversation

benissimo
Copy link

This adds support for ECDHE-RSA-AES128-GCM-SHA256. Many more are still missing,
but I am not familiar enough with openssl to go through and configure them.

Update
I've amended this commit, removing the jopenssl.tar.gz file.
I've also rebased this branch onto the latest master from
jruby/jruby-openssl.

This adds support for ECDHE-RSA-AES128-GCM-SHA256. Many more are still missing,
but I am not familiar enough with openssl to go through and configure them.

*Update*
I've amended this commit, removing the jopenssl.tar.gz file.
I've also rebased this branch onto the latest master from
jruby/jruby-openssl.
@benissimo
Copy link
Author

Travis CI build failed. I'm not familiar with the build's setup and requirements.

@spuder
Copy link

spuder commented Sep 9, 2019

The travis errors appear to be similar to errors other have reported here: https://travis-ci.community/t/openjdk-7-no-longer-installs-on-xenial/4012

@amo13
Copy link

amo13 commented Sep 26, 2019

This would probably resolve #59

@ThomasKoppensteiner
Copy link

Can someone please fix the Travis CI build, so that this PR can be merged?

@kares
Copy link
Member

kares commented Feb 4, 2020

support for ECDHE-RSA-AES128-GCM-SHA256 (and other GCMs) is now on master
there's still issues when trying to test out these work but expect this to lend in 0.10.3
the problem with PRs such as this is that they need some way of verification/testing

@kares kares closed this Feb 4, 2020
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.

5 participants