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: `Request::path(&self)`: show `/` to user if internal path bytes is `` (empty) #74

Merged

kanarus merged 2 commits into main from fix_request_path_handling

Feb 5, 2024

Member

kanarus commented Feb 5, 2024

Add a comment describing why it's ok to force-strip trailing / in request path even if raw path was just /
Fix: Request::path(&self) to show user / if internal path bytes is `` (empty, e.t. the original request path was /)

kanarus added 2 commits

February 6, 2024 01:36


          Fix: Request::path(&self): show / to user if internal path bytes …

5f03895

…is `` (empty)


          use get_unchecked in striping '/'

a932cd7

kanarus merged commit a88bde9 into main

16 checks passed

kanarus deleted the fix_request_path_handling branch

February 5, 2024 16:47

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

Labels

None yet