Redirects fail if URL has appended query string
|Reported by:||richardb||Owned by:||nobody|
|Cc:||Triage Stage:||Design decision needed|
|Has patch:||no||Needs documentation:||no|
|Needs tests:||yes||Patch needs improvement:||no|
The redirect middleware obtains the path of the current request with path = request.get_full_path(), then tries to find a match in the Redirect table.
If there is an appended query string, it will include this when querying the Redirect table (and so fail to find a match).
Shouldn't we use path = request.path instead?
Change History (3)
comment:1 Changed 5 years ago by dmoisset
- Needs documentation unset
- Needs tests set
- Patch needs improvement unset
- Triage Stage changed from Unreviewed to Accepted
comment:2 Changed 5 years ago by dmoisset
- Triage Stage changed from Accepted to Design decision needed