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

webserver: somewhat parse passed host before using #27

Merged
merged 1 commit into from
Apr 8, 2014

Conversation

asfaltboy
Copy link
Contributor

This uses standard urlparse package to attempt to parse a url scheme before falling back to the raw string passed.

Close Issue #15

This uses standard urlparse package to attempt to parse a url scheme before falling back to the raw string passed.

Close Issue FiloSottile#16
@asfaltboy asfaltboy mentioned this pull request Apr 8, 2014
@FiloSottile
Copy link
Owner

Thanks! However the code in use now is the bleed_serve.go.
It was simply too much for web.py

asfaltboy added a commit to asfaltboy/Heartbleed that referenced this pull request Apr 8, 2014
Same as pull request FiloSottile#27 only for the go webserver

Close issue FiloSottile#15 again

(please bear with me, I never did write go in my life)
@asfaltboy
Copy link
Contributor Author

Here's another try: PR #30#30

On Tue, Apr 8, 2014 at 10:25 PM, Filippo Valsorda
notifications@github.comwrote:

Thanks! However the code in use now is the bleed_serve.go.
It was simply too much for web.py

Reply to this email directly or view it on GitHubhttps://github.com//pull/27#issuecomment-39890696
.

@FiloSottile
Copy link
Owner

Good, merged the other, let's merge this too just not to leave it laying around :)

FiloSottile added a commit that referenced this pull request Apr 8, 2014
webserver: somewhat parse passed host before using
@FiloSottile FiloSottile merged commit e5a525e into FiloSottile:master Apr 8, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants