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

No existing session to resume #97

Open
dgolombek opened this issue Jun 21, 2016 · 2 comments
Open

No existing session to resume #97

dgolombek opened this issue Jun 21, 2016 · 2 comments

Comments

@dgolombek
Copy link
Contributor

dgolombek commented Jun 21, 2016

Environment

  • jruby-openssl 0.9.17
  • net-http-persistent 2.9.4

Expected Behavior

Opening a new session should work.

Actual Behavior

No existing session to resume: org/jruby/ext/openssl/SSLSocket.java:217:in `connect'
/tmp/jetty-0.0.0.0-8080-evergreen.war-_evergreen-any-412843000765214245.dir/webapp/WEB-INF/gems/gems/net-http-persistent-2.9.4/lib/net/http/persistent/ssl_reuse.rb:70:in `connect'
org/jruby/ext/timeout/Timeout.java:115:in `timeout'
/tmp/jetty-0.0.0.0-8080-evergreen.war-_evergreen-any-412843000765214245.dir/webapp/WEB-INF/gems/gems/net-http-persistent-2.9.4/lib/net/http/persistent/ssl_reuse.rb:70:in `connect'
/tmp/jetty-0.0.0.0-8080-evergreen.war-_evergreen-any-412843000765214245.dir/webapp/WEB-INF/lib/jruby-stdlib-1.7.25.jar!/META-INF/jruby.home/lib/ruby/1.9/net/http.rb:756:in `do_start'
/tmp/jetty-0.0.0.0-8080-evergreen.war-_evergreen-any-412843000765214245.dir/webapp/WEB-INF/lib/jruby-stdlib-1.7.25.jar!/META-INF/jruby.home/lib/ruby/1.9/net/http.rb:751:in `start'
/tmp/jetty-0.0.0.0-8080-evergreen.war-_evergreen-any-412843000765214245.dir/webapp/WEB-INF/gems/gems/net-http-persistent-2.9.4/lib/net/http/persistent.rb:700:in `start'
/tmp/jetty-0.0.0.0-8080-evergreen.war-_evergreen-any-412843000765214245.dir/webapp/WEB-INF/gems/gems/net-http-persistent-2.9.4/lib/net/http/persistent.rb:965:in `reset'
/tmp/jetty-0.0.0.0-8080-evergreen.war-_evergreen-any-412843000765214245.dir/webapp/WEB-INF/gems/gems/net-http-persistent-2.9.4/lib/net/http/persistent.rb:628:in `connection_for'
/tmp/jetty-0.0.0.0-8080-evergreen.war-_evergreen-any-412843000765214245.dir/webapp/WEB-INF/gems/gems/net-http-persistent-2.9.4/lib/net/http/persistent.rb:994:in `request'
/tmp/jetty-0.0.0.0-8080-evergreen.war-_evergreen-any-412843000765214245.dir/webapp/WEB-INF/gems/gems/faraday-0.9.2/lib/faraday/adapter/net_http_persistent.rb:25:in `perform_request'
/tmp/jetty-0.0.0.0-8080-evergreen.war-_evergreen-any-412843000765214245.dir/webapp/WEB-INF/gems/gems/faraday-0.9.2/lib/faraday/adapter/net_http.rb:40:in `call'
/tmp/jetty-0.0.0.0-8080-evergreen.war-_evergreen-any-412843000765214245.dir/webapp/WEB-INF/gems/gems/faraday-0.9.2/lib/faraday/adapter/net_http_persistent.rb:21:in `with_net_http_connection'
/tmp/jetty-0.0.0.0-8080-evergreen.war-_evergreen-any-412843000765214245.dir/webapp/WEB-INF/gems/gems/faraday-0.9.2/lib/faraday/adapter/net_http.rb:32:in `call'

Calls from up in Faraday and higher elided. This started after upgrading from 0.9.16 to 0.9.17 and went away as soon as we reverted. It happened on every session that was created. I believe this was caused by ed95b2c

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/35343015-no-existing-session-to-resume?utm_campaign=plugin&utm_content=tracker%2F136995&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F136995&utm_medium=issues&utm_source=github).
@kares
Copy link
Member

kares commented Jun 21, 2016

would appreciate an isolated .rb reproduction case - to confirm the actual code is not assuming to resume a non existing session (or to see if ed95b2c can get tuned some way to handle the case). thanks!

@dgolombek
Copy link
Contributor Author

Yep, will do -- sorry about the poor bug report yesterday, I was rushing. I
suspect this is a bad interaction between Faraday, Net::Http::Persistent,
and jruby-openssl. This is a heavily threaded app that's making a lot of
connections to a lot of different endpoints -- hopefully the testcase will
be simpler.

Thanks
Dave

On Tue, Jun 21, 2016 at 3:49 AM, Karol Bucek notifications@github.com
wrote:

would appreciate an isolated .rb reproduction case - to confirm the actual
code is not assuming to resume a non existing session (or to see if
ed95b2c
ed95b2c
can get tuned some way to handle the case). thanks!


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#97 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AAVDqUfidgP7sv9oZrFgAZ9-z_lDqvbcks5qN5efgaJpZM4I6RaV
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants