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 bug #67587: Redirection loop on nginx with FPM #712

Merged
merged 1 commit into from Jul 8, 2014

Conversation

4 participants
@cweiske
Contributor

cweiske commented Jul 8, 2014

Redirection will work correctly as long as PATH_INFO is available,
e.g. via the following nginx configuration:

set $path_info $fastcgi_path_info;
fastcgi_param PATH_INFO $path_info;
Fix bug #67587: Redirection loop on nginx with FPM
Redirection will work correctly as long as PATH_INFO is available,
e.g. via the following nginx configuration:

    set $path_info $fastcgi_path_info;
    fastcgi_param PATH_INFO $path_info;
@Tyrael

This comment has been minimized.

Show comment
Hide comment
@Tyrael

Tyrael Jul 8, 2014

Member

the patch seems trivial.
@jpauli @smalyshev should we merge this into 5.4/5.5 or should I only merge it into 5.6 (and master) and we cherrypick it into 5.4/5.5 later?

Member

Tyrael commented Jul 8, 2014

the patch seems trivial.
@jpauli @smalyshev should we merge this into 5.4/5.5 or should I only merge it into 5.6 (and master) and we cherrypick it into 5.4/5.5 later?

@smalyshev

This comment has been minimized.

Show comment
Hide comment
@smalyshev

smalyshev Jul 8, 2014

Contributor

@Tyrael looks ok for 5.4

Contributor

smalyshev commented Jul 8, 2014

@Tyrael looks ok for 5.4

@Tyrael

This comment has been minimized.

Show comment
Hide comment
@Tyrael

Tyrael Jul 8, 2014

Member

@smalyshev awesome, will you merge the PR or should I?

Member

Tyrael commented Jul 8, 2014

@smalyshev awesome, will you merge the PR or should I?

@php-pulls php-pulls merged commit e76d3cb into php:master Jul 8, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment