Notebook redirects use wrong HTTP Status Code (301 Moved Permanently)

[TimDumol]

Notebook redirects use status code 301 to redirect to new pages, when they should use code 303 See Other. Because of this, Google Chrome caches the redirect, leading to problems with creating new worksheets, emptying trash, etc. If the redirect is cached, the requested action will not be performed, as the browser will redirect directly to the original url.

For example, if you are to click the "New Worksheet" link twice, you would expect to create two new worksheets. However, the second click redirects you directly to the first created worksheet.

Thanks to mpatel for spotting this.

CC: @qed777

Component: notebook

Author: Tim Dumol, Alex Leone

Reviewer: William Stein

Merged: sage-4.4.1.alpha2

Issue created by migration from https://trac.sagemath.org/ticket/8724

[TimDumol]

Attachment: trac_8724-sagenb-redirect-code.patch.gz

Changes status code of redirects to 303. Also fixes a bunch of Selenium tests. Apply to sagenb repo.

[TimDumol]

Author: Tim Dumol

[jhpalmieri]

comment:2

(I'm not changing any notebook code in Sage 4.4.)

[williamstein]

comment:3

This looks great!

[sagetrac-acleone]

Apply after first patch

[jhpalmieri]

comment:5

Attachment: trac_8724-doctest-fix.patch.gz

[williamstein]

Merged: 4.4.1.alpha2

[sagetrac-mvngu]

Reviewer: William Stein

[sagetrac-mvngu]

Changed author from Tim Dumol to Tim Dumol, Alex Leone

[sagetrac-mvngu]

Changed merged from 4.4.1.alpha2 to sage-4.4.1.alpha2