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.

Sign up for GitHub

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

Jump to bottom

Fix a looping regression for 303 responses without a Location header #342

Merged
merged 2 commits into from Jun 18, 2020
Merged

Fix a looping regression for 303 responses without a Location header #342

merged 2 commits into from Jun 18, 2020

Commits on Jun 15, 2020

  1. Fix a looping regression for 303 responses without a Location header 

    The change in 6.45 had a side effect of making it look at max_redirect
only when the Location header is set.

The libapache2-mod-perl2 test suite implements a test server that answers
with HTTP code 303 without including such a header.

RFC 7231 is not very clear on whether Location is required, although
the response doesn't make much sense without it.

Some googling reveals this is not a new question.

  https://stackoverflow.com/questions/16194988/for-which-3xx-http-codes-is-the-location-header-mandatory

For the sake of robustness I suppose libwww-perl needs to be fixed.

Bug-Debian: https://bugs.debian.org/962904
    @ntyni
    ntyni committed Jun 15, 2020
    Configuration menu
    Copy the full SHA
    789ba04 View commit details
    Browse the repository at this point in the history

Commits on Jun 17, 2020

  1. Update Changes to placate Travis CI

    @ntyni
    ntyni committed Jun 17, 2020
    Configuration menu
    Copy the full SHA
    a8ae2b7 View commit details
    Browse the repository at this point in the history