8000 URI parsing fails due to lack of host in parse_url · Issue #29630 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

URI parsing fails due to lack of host in parse_url #29630

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
Closed
ian-starts opened this issue Dec 17, 2018 · 1 comment
Closed

URI parsing fails due to lack of host in parse_url #29630

ian-starts opened this issue Dec 17, 2018 · 1 comment
Labels
Bug HttpFoundation Status: Needs Review

Comments

@ian-starts
Copy link
ian-starts commented Dec 17, 2018 •
edited
Loading

Symfony version(s) affected: 4.3

Description
URL parsing fails. For example:
https://nova.laravel.com/password/reset
This resolves correctly.
However https://nova.laravel.com//password/reset this results in a 404. Which could be seen as a user error.
But https://nova.laravel.com//foobar/password/reset also resolves, however incorrectly. Clearly signaling it as a bug.

How to reproduce
Go to https://nova.laravel.com//foobar/password/reset. Or in any Laravel/Symfony project add an extra / after the host.

Possible Solution
I added a possible solution in this PR.
#29631
@nicolas-grekas
Copy link
Member

Duplicate of #29478

@nicolas-grekas nicolas-grekas marked this as a duplicate of #29478 Dec 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug HttpFoundation Status: Needs Review
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@nicolas-grekas @xabbuh @carsonbot @ian-starts
0