You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Since tower-http/src/normalize_path.rs has already gone in the other direction, I guess the best way forward would be to make it configurable so that those who prefer trailing slashes can get them?
I'll happily make the change.
The text was updated successfully, but these errors were encountered:
The existing constructor is named trim_trailing_slash (rather than new) to allow for other constructors / normalization styles to be added down the line. So feel free to send a PR. I think I had a short discussion with somebody around what to name the "add trailing slash if missing" style before, but I don't know where that was. If it's not in another issue or GH discussion, it's probably going to be impossible to find. Settling the exact name can happen on an otherwise finished PR, IMHO.
I think paths should be normalised to end with a slash.
See how (and why) Django does this:
Since
tower-http/src/normalize_path.rs
has already gone in the other direction, I guess the best way forward would be to make it configurable so that those who prefer trailing slashes can get them?I'll happily make the change.
The text was updated successfully, but these errors were encountered: