Opened 11 years ago

Closed 11 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: dev
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

Description

In https://docs.djangoproject.com/en/dev/ref/contrib/redirects/ 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 by Baptiste Mispelon, 11 years ago

Triage Stage: UnreviewedAccepted

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

comment:2 by Bouke Haarsma, 11 years ago

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

comment:3 by Claude Paroz, 11 years ago

Triage Stage: AcceptedReady for checkin

comment:4 by Tim Graham <timograham@…>, 11 years ago

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