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

Conform to the RFC for "forwarded" header #106

Merged
merged 1 commit into from Sep 27, 2016
Merged

Conform to the RFC for "forwarded" header #106

merged 1 commit into from Sep 27, 2016

Conversation

@patrickkusebauch
Copy link
Contributor

patrickkusebauch commented Sep 27, 2016

  • bugfix
  • issues - #101
  • documentation - not needed
  • BC break - yes

BC break if and only if somebody implemented their proxy settings by the actual code. If they went by the documentation or the RFC, they should be fine and this commit actually fixes things.

@dg dg merged commit b9257f9 into nette:master Sep 27, 2016
3 checks passed
3 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 79.748%
Details
@patrickkusebauch patrickkusebauch deleted the patrickkusebauch:forwarded-bug branch Sep 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.