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 #2331 -- fix folder handling/redirects in auto #2336

Merged
merged 1 commit into from May 5, 2016
Merged

Fix #2331 -- fix folder handling/redirects in auto #2336

merged 1 commit into from May 5, 2016

Conversation

@Kwpolska
Copy link
Member

@Kwpolska Kwpolska commented May 5, 2016

This patch:

  • works on URLs and not file system path (was messing up Windows)
  • uses canonical HTTP status description

Signed-off-by: Chris Warrick kwpolska@gmail.com

Signed-off-by: Chris Warrick <kwpolska@gmail.com>
@ralsina
Copy link
Member

@ralsina ralsina commented May 5, 2016

LGTM

@ralsina ralsina merged commit 8f58062 into master May 5, 2016
4 checks passed
4 checks passed
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@ralsina ralsina deleted the fix-2331 branch May 5, 2016
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

2 participants