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

Errno::EINVAL when SSL is enabled and Firefox does not trust self-signed cert #1564

Closed
perlun opened this issue Apr 18, 2018 · 2 comments
Closed

Comments

@perlun
Copy link
Contributor

perlun commented Apr 18, 2018

Steps to reproduce

  1. Use a self-signed certificate with Puma.
  2. Browse to the https://localhost:12345 port.
  3. Firefox will show a security warning, asking you to add an exception.

Expected behavior

The request should be processed without errors.

Actual behavior

Error in reactor loop escaped: Invalid argument - getpeername(2) (Errno::EINVAL)
/Users/plundberg/.rvm/gems/ruby-2.5.1/gems/puma-3.11.3/lib/puma/minissl.rb:155:in `peeraddr'
/Users/plundberg/.rvm/gems/ruby-2.5.1/gems/puma-3.11.3/lib/puma/minissl.rb:155:in `peeraddr'
/Users/plundberg/.rvm/gems/ruby-2.5.1/gems/puma-3.11.3/lib/puma/reactor.rb:93:in `rescue in block in run_internal'
/Users/plundberg/.rvm/gems/ruby-2.5.1/gems/puma-3.11.3/lib/puma/reactor.rb:73:in `block in run_internal'
/Users/plundberg/.rvm/gems/ruby-2.5.1/gems/puma-3.11.3/lib/puma/reactor.rb:43:in `each'
/Users/plundberg/.rvm/gems/ruby-2.5.1/gems/puma-3.11.3/lib/puma/reactor.rb:43:in `run_internal'
/Users/plundberg/.rvm/gems/ruby-2.5.1/gems/puma-3.11.3/lib/puma/reactor.rb:154:in `block in run_in_thread'

This is mostly a minor annoyance; after adding the exception it seems to work fine.

System configuration

Ruby version: 2.5.1
Rails version: N/A
Puma version: 3.11.3

@sevenseacat
Copy link

I added the security exception in Firefox and I'm still seeing the error in my logs.

Ruby version: 2.5.1
Rails version: 5.1
Puma version: 3.12.0

@sevenseacat
Copy link

False alarm - there were a few other domains I needed to add security exceptions for as well!

This was referenced Mar 9, 2021
This was referenced Mar 14, 2021
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

3 participants