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.

Sign up for GitHub

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

Jump to bottom

Always use TLS with HTTP20Connection #58

Merged

Pr0Ger merged 1 commit into Pr0Ger:master from PaulosV:fix_alt_port_insecure_connection

Dec 18, 2017

Contributor

PaulosV commented Dec 12, 2017 •

edited

Loading

This snippet fixes usage of alternative port (2197) of HTTP/2 APN service.

hyper package only uses TLS by default if the port is 443. By adding secure=True, we make it use TLS in all circumstances.

http://hyper.readthedocs.io/en/latest/api.html#hyper.HTTP20Connection


          Always use TLS with HTTP20Connection

1c1299f

PaulosV mentioned this pull request

Connection with alternative port fails #59

Closed

Pr0Ger approved these changes

View reviewed changes

Pr0Ger merged commit d071ec1 into Pr0Ger:master

Owner

Pr0Ger commented Dec 18, 2017

Thx for contributions, @PaulosV

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