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

Fix #1780 #2167

Merged
merged 1 commit into from May 29, 2018
Merged

Fix #1780 #2167

merged 1 commit into from May 29, 2018

Conversation

@tchaloupka
Copy link
Contributor

@tchaloupka tchaloupka commented May 28, 2018

Tried to fix this. It works ok for me with this change.

@s-ludwig
Copy link
Member

@s-ludwig s-ludwig commented May 29, 2018

Looks like a workable solution, I'd just mention in a comment that this is a workaround for non-standard-conformant servers and #1780 in particular.

@tchaloupka tchaloupka force-pushed the tchaloupka:fix_1780 branch from 2cb48de to 446a578 May 29, 2018
@tchaloupka
Copy link
Contributor Author

@tchaloupka tchaloupka commented May 29, 2018

Thx, adjusted the comment

@s-ludwig
Copy link
Member

@s-ludwig s-ludwig commented May 29, 2018

Okay, thanks! Set to merge.

@dlang-bot dlang-bot merged commit b65b77c into vibe-d:master May 29, 2018
3 checks passed
3 checks passed
codecov/patch 25% of diff hit (target 59.554%)
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants