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

URLs with different numbers of trailing slashes #616

Closed
mkduew opened this issue Jan 6, 2025 · 3 comments
Closed

URLs with different numbers of trailing slashes #616

mkduew opened this issue Jan 6, 2025 · 3 comments
Labels

Comments

@mkduew
Copy link

mkduew commented Jan 6, 2025

The Flight PHP Framework treats URLs with different numbers of trailing slashes (/) as distinct routes instead of normalizing them. For example, the following requests result in different outcomes or errors:

Expected Behavior:
Multiple trailing slashes in a URL should be normalized to a single slash to ensure consistent behavior, as per common web server and framework standards.

@mkduew mkduew added the Bug label Jan 6, 2025
@n0nag0n
Copy link
Collaborator

n0nag0n commented Jan 9, 2025

Just curious, how did you discover this? Just messing around or is your app redirecting in funky ways?

@mkduew
Copy link
Author

mkduew commented Jan 9, 2025

It was a coincidence. I was messing around.

@mkduew mkduew closed this as completed Jan 9, 2025
@mkduew mkduew reopened this Jan 9, 2025
@n0nag0n
Copy link
Collaborator

n0nag0n commented Jan 14, 2025

Fixed in new release. https://github.com/flightphp/core/releases/tag/v3.13.1

Kinda pointless to fix, but fun I suppose!

@n0nag0n n0nag0n closed this as completed Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants