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

Parse REQUEST_URI properly #1682

Merged
merged 1 commit into from Dec 17, 2015

Conversation

Projects
None yet
2 participants
@akrabat
Member

akrabat commented Dec 16, 2015

parse_url() requires a full URL. As we don't extract the domain name
or scheme from this, we can use a stand-in. This allows the REQUEST_URI
to start with multiple forward slashes...

Fixes #1679

Parse REQUEST_URI properly
parse_url() requires a full URL. As we don't extract the domain name
or scheme from this, we can use a stand-in. This allows the REQUEST_URI
to start with multiple forward slashes...

Fixes #1679

@akrabat akrabat added the Slim 3 label Dec 16, 2015

@akrabat akrabat added this to the 3.1.0 milestone Dec 16, 2015

codeguy added a commit that referenced this pull request Dec 17, 2015

@codeguy codeguy merged commit e88c825 into slimphp:3.x Dec 17, 2015

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 89.668%
Details

rivetchip pushed a commit to rivetchip/Slim that referenced this pull request Jan 22, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment