-
-
Notifications
You must be signed in to change notification settings - Fork 109
disable SSL Inspection #39
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
Conversation
|
This pull request adds two features:
The CI isn't working correctly but i don't know if that matters. |
|
@jeeftor the build is failing in Python 3.3 only -- perhaps you need to include some compatibility code? |
|
Also, I'd prefer if you submitted the hyphen-joining code in a separate PR, if that wouldn't be too much of a hassle :) |
|
I'm ok with the idea, however, it could use a better wording than |
|
@waldyrious Do we proceed with merging this one? |
Signed-off-by: Matthew Peveler <[email protected]>
|
@zdroid @waldyrious @felixonmars I updated the PR for the requested changes and fixed merge conflicts and I think this should be good to go? |
|
Good work here! |
Closes #11.