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

avoiding ssl.session= on JRuby #72

Open
kares opened this issue Mar 29, 2016 · 4 comments
Open

avoiding ssl.session= on JRuby #72

kares opened this issue Mar 29, 2016 · 4 comments

Comments

@kares
Copy link

kares commented Mar 29, 2016

its just a stub since we can not implement session reuse using the underlying Java security APIs.

users seems to be getting annoyed by our session= printing a warning.

would that be acceptable if that session= line did a unless defined? JRUBY_VERSION check?

@drbrain
Copy link
Owner

drbrain commented Mar 29, 2016

Yes, that would be fine

@kares
Copy link
Author

kares commented Apr 1, 2016

did not rush into this as I stumbled upon an experiment: jruby/jruby-openssl@f12aab2 in jruby-openssl ... where setting the session = would try to force session re-use on the Java end ... not exactly swapping the session but probably gets as close as we can. unfortunately this would need some real world testing if its behaving as expected.

@lolwhitaker would you be interested in confirming it works as expected (since you fired up a PR) ?

@lolwhitaker
Copy link

@kares Happy to help with testing if you can guide me a little as to the different scenarios (both positive and negative) you think should be covered.

@kares
Copy link
Author

kares commented Apr 21, 2016

@lolwhitaker you could try jruby-openssl 0.9.17 pre-release from: https://oss.sonatype.org/content/repositories/snapshots/rubygems/jruby-openssl/0.9.17.dev-SNAPSHOT/ ... did some testing using bundle install --jobs xx (from jruby/jruby#3635) WARNING: SSLSocket#session= is not supported are gone and things seems to work. the change is that session= is now trying to force the underlying engine to not create new sessions but re-use. this needs real-world testing (not yet sure if it will be in 0.9.17)

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 a pull request may close this issue.

3 participants