Opened 6 years ago

Closed 6 years ago

#21354 closed Bug (fixed)

'contrib.redirects' documentation doesn't say whether it's a 301 or 302

Reported by: German Larrain Owned by: Bouke Haarsma
Component: contrib.redirects Version: master
Severity: Normal Keywords: documentation
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no


In it should be said that the redirects are permanent (i.e. 301), not 302. Although one could guess that's the reasonable behavior, better make it explicit. On the other hand, at the end of the page there is a mention of how to change the response classes used, and that the default value of RedirectFallbackMiddleware.response_redirect_class is HttpResponsePermanentRedirect, so it could be inferred that the default behavior is return a 301 but again, not explicit enough.

Change History (4)

comment:1 Changed 6 years ago by Baptiste Mispelon

Triage Stage: UnreviewedAccepted

I agree, this should be mentionned more prominently (maybe even in the first paragraph).

comment:2 Changed 6 years ago by Bouke Haarsma

Has patch: set
Owner: changed from nobody to Bouke Haarsma
Status: newassigned

comment:3 Changed 6 years ago by Claude Paroz

Triage Stage: AcceptedReady for checkin

comment:4 Changed 6 years ago by Tim Graham <timograham@…>

Resolution: fixed
Status: assignedclosed

In 073b974e6613adc88cade7d34af701e52e2cc8fd:

Fixed #21354 -- Documented contrib.redirects defaults to 301 redirects

Thanks glarrain for the suggestion.

Note: See TracTickets for help on using tickets.
Back to Top