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

Ethon::Errors::InvalidOption: The option: disable_ssl_peer_verification is invalid #11

Open
bradwheel opened this issue Aug 4, 2016 · 6 comments

Comments

@bradwheel
Copy link

bradwheel commented Aug 4, 2016

We're receiving this error, which seems to be traced back to this gem's use of Typhoeus:

Ethon::Errors::InvalidOption:
       The option: disable_ssl_peer_verification is invalid.
       Please try ssl_verifypeer instead of disable_ssl_peer_verification.

We've been unable to reproduce this locally, but is breaking all of our builds on CI. After looking into the issue, it's pretty common and is related to some use of the faraday typhoeus adapter. This is especially relevant for us given that we use elasticsearch-ruby, which uses typhoeus and the faraday adapter.

Error documented here:
swagger-api/swagger-codegen#3380
typhoeus/typhoeus#270
elastic/elasticsearch-ruby#72

@dlangevin
Copy link
Contributor

It looks like you can load their monkey patch (maybe in an initializer) and
that should fix it

require 'typhoeus/adapters/faraday'

On Thu, Aug 4, 2016 at 5:59 PM bradwheel [email protected] wrote:

We're receiving this error, which seems to be traced back to this gem's
use of Typhoeus:

Ethon::Errors::InvalidOption:
The option: disable_ssl_peer_verification is invalid.
Please try ssl_verifypeer instead of disable_ssl_peer_verification.

We've been unable to reproduce this locally, but is breaking all of our
builds on CI. After looking into the issue, it's pretty common and is
related to some use of the faraday typhoeus adapter. This is especially
relevant for us given that we use elasticsearch-ruby, which uses typhoeus
and the faraday adapter.

Error documented here:
typhoeus/typhoeus#270 typhoeus/typhoeus#270
elastic/elasticsearch-ruby#72
elastic/elasticsearch-ruby#72


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#11, or mute the thread
https://github.com/notifications/unsubscribe-auth/AADjt49suXDEcJ3DF3dRjKQ1AFgHqrz7ks5qcmC6gaJpZM4JdKgY
.

@dlangevin
Copy link
Contributor

Another suggestion would be to wrap it in an adapter and stub it in test.

On Thu, Aug 4, 2016 at 6:18 PM Dan Langevin [email protected] wrote:

It looks like you can load their monkey patch (maybe in an initializer)
and that should fix it

require 'typhoeus/adapters/faraday'

On Thu, Aug 4, 2016 at 5:59 PM bradwheel [email protected] wrote:

We're receiving this error, which seems to be traced back to this gem's
use of Typhoeus:

Ethon::Errors::InvalidOption:
The option: disable_ssl_peer_verification is invalid.
Please try ssl_verifypeer instead of disable_ssl_peer_verification.

We've been unable to reproduce this locally, but is breaking all of our
builds on CI. After looking into the issue, it's pretty common and is
related to some use of the faraday typhoeus adapter. This is especially
relevant for us given that we use elasticsearch-ruby, which uses typhoeus
and the faraday adapter.

Error documented here:
typhoeus/typhoeus#270 typhoeus/typhoeus#270
elastic/elasticsearch-ruby#72
elastic/elasticsearch-ruby#72


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#11, or mute the thread
https://github.com/notifications/unsubscribe-auth/AADjt49suXDEcJ3DF3dRjKQ1AFgHqrz7ks5qcmC6gaJpZM4JdKgY
.

@bradwheel
Copy link
Author

We tried the faraday adapter and were unsuccessful in getting it to run. I spent some effort over the weekend looking into it and I think elasticsearch-ruby is the culprit here. I'll keep investigating as time allows.

@sameerr25
Copy link

Any update here?

@dlangevin
Copy link
Contributor

@sameerr25 we have a branch where we switched the underlying adapter to be HTTPClient rather than Typhoeus (which was causing an issue with elasticsearch-ruby) Will be pushing a new version shortly

@william-beange-hs
Copy link

monkey patch worked for me... require 'typhoeus/adapters/faraday' seems sketch though

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

No branches or pull requests

4 participants