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

Force SSLv3. #1850

Closed
wants to merge 1 commit into from
Closed

Force SSLv3. #1850

wants to merge 1 commit into from

Conversation

Lukasa
Copy link
Member

@Lukasa Lukasa commented Jan 7, 2014

This is a possible fix for #1847.

@t-8ch
Copy link
Contributor

t-8ch commented Jan 7, 2014

According to the table on http://docs.python.org/2/library/ssl.html#ssl.wrap_socket this is going to blow up on more mordern servers. (At least it is supposed to)

@Lukasa
Copy link
Member Author

Lukasa commented Jan 7, 2014

@Lukasa
Copy link
Member Author

Lukasa commented Jan 7, 2014

Based on my reading of that table, selecting SSLv3 will work on servers that don't offer SSLv2, but SSLv23 will not work. Conversely, SSLv3 will not work on servers that don't offer anything anything better than SSLv2, and the prevailing attitude seems to be "who cares about those people".

@t-8ch
Copy link
Contributor

t-8ch commented Jan 7, 2014

Please try: https://t-8ch.de I disabled everything except TLS.
Using requests with SSLv3 doesn't work for me.

@Lukasa
Copy link
Member Author

Lukasa commented Jan 7, 2014

Yup, you're right. Goddamn I hate SSL so much.

@Lukasa Lukasa closed this Jan 7, 2014
@t-8ch
Copy link
Contributor

t-8ch commented Jan 7, 2014

And SSLv23 works with all version of python (still openssl 1.0.1f)

@Lukasa
Copy link
Member Author

Lukasa commented Jan 7, 2014

Doesn't with old versions of OpenSSL though.

@Lukasa Lukasa deleted the goddamnssl branch January 7, 2014 20:52
@t-8ch
Copy link
Contributor

t-8ch commented Jan 7, 2014

Those probably don't support TLS at all

@sigmavirus24
Copy link
Contributor

Thanks @t-8ch

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants