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

Fix connection.verify_mode for Net::HTTP https connection to VERIFY_PEER #55

Open
kaiwren opened this issue Dec 9, 2010 · 7 comments
Open

Comments

@kaiwren
Copy link
Contributor

kaiwren commented Dec 9, 2010

Also, make it configurable. See http://www.rubyinside.com/how-to-cure-nethttps-risky-default-https-behavior-4010.html

@nikhilvallishayee
Copy link
Contributor

I have defaulted the verify_mode to VERIFY_PEER while creating a connection and made it configurable by being able to pass an options hash to the create method specifying the verify_mode

@kaiwren
Copy link
Contributor Author

kaiwren commented Dec 16, 2010

Remember to update the changelog once this is complete.

@jasim
Copy link
Contributor

jasim commented Dec 27, 2010

  1. All HTTPS requests are VERIFY_PEER. The option to choose VERIFY_NONE is not exposed to the Uri object.

  2. VERIFY_MODE value is being expected as a string:
    if options[:verify_mode] == 'VERIFY_NONE'

    It should ideally be the symbol OpenSSL::SSL::VERIFY_PEER itself.

@nikhilvallishayee
Copy link
Contributor

Fixed and pushed a week back. Hasn't been updated here for unknown reason. Redid push. Still not updated!

@kaiwren
Copy link
Contributor Author

kaiwren commented Jan 2, 2011

What's happening on this one? Nik, could you link to the commits that you're talking about? Also, did you see my comments on the use of constants on 92a6acf ?

@nikhilvallishayee
Copy link
Contributor

done!
kaiwren@c74c020

@kaiwren
Copy link
Contributor Author

kaiwren commented Jan 5, 2011

Nik, could you also take a look at #66? It seems to be a consequence of switching to always verifying certificates.

@kaiwren kaiwren closed this as completed Nov 19, 2015
@kaiwren kaiwren reopened this Nov 19, 2015
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