302s can also wind up cached in some browsers, which is why the new 307 was invented, but 307 Temporary Redirect is also more correct (imo).

CommentFileSizeAuthor
#4 interdiff_2_4.txt309 bytesnixou
#4 r4032login.3065998.add-307-temp-redirect-option.4.patch1.74 KBnixou
#2 r4032login.3065998.add-307-temp-redirect-option.patch1.32 KBgrayle

Comments

Grayle created an issue. See original summary.

grayle’s picture

grayle commented
Status: Active » Needs review
StatusFileSize
new r4032login.3065998.add-307-temp-redirect-option.patch1.32 KB

Included is a patch which adds 307 to the select list and sets it as the default when installing the module.

tim.anderson’s picture

tim.anderson commented
Status: Needs review » Reviewed & tested by the community

Simple code, and it works fine. Thanks!

nixou’s picture

nixou commented
StatusFileSize
new r4032login.3065998.add-307-temp-redirect-option.4.patch1.74 KB
new interdiff_2_4.txt309 bytes

Updated with tests.

  • Nixou committed 9090436 on 8.x-1.x authored by Grayle 
    Issue #3065998 by Nixou, Grayle: Allow for 307 Temporary Redirect
nixou’s picture

nixou commented
Status: Reviewed & tested by the community » Fixed

Committed, thank you !

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.