Skip to content

Allow Server Name Identification to be set #62

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

Merged
merged 4 commits into from
Oct 8, 2017
Merged

Conversation

timkent
Copy link
Collaborator

@timkent timkent commented Oct 7, 2017

Closes #49


DEFAULT_USER_AGENT = 'Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/61.0.3163.100 Safari/537.36'

_target_host = None
_ssl_wrap_socket = ssl_.ssl_wrap_socket
def ssl_wrap_socket(sock, keyfile=None, certfile=None, cert_reqs=None,
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should move this into the class. Going to merge for now since it's a bugfix but I think we should definitely do some refactoring on this file soon.

Copy link
Owner

@codingo codingo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should move this into the class. Going to merge for now since it's a bugfix but I think we should definitely do some refactoring on this file soon.

@codingo codingo merged commit b2ec938 into master Oct 8, 2017
@codingo codingo deleted the timk-sni-monkey-patch branch October 8, 2017 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants