Merge pull request #181 from arran4/fix-redirect-405-error-7743134787219650796

Fix 405 Method Not Allowed error on redirect after POST

Author: arran4

autoRefreshPage.go

@@ -27,5 +27,5 @@ func TaskDoneAutoRefreshPage(w http.ResponseWriter, r *http.Request) error {
 func taskRedirectWithoutQueryArgs(w http.ResponseWriter, r *http.Request) {
 	u := r.URL
 	u.RawQuery = ""
-	http.Redirect(w, r, u.String(), http.StatusTemporaryRedirect)
+	http.Redirect(w, r, u.String(), http.StatusSeeOther)
 }

bookmarkActionHandlers.go

@@ -33,7 +33,7 @@ func BookmarksEditSaveAction(w http.ResponseWriter, r *http.Request) error {
 			if p := providerFromContext(r.Context()); p != nil {
 				if err := p.CreateRepo(r.Context(), login, token, repoName); err == nil {
 					if err := CreateBookmarks(r.Context(), login, token, branch, text); err == nil {
-						http.Redirect(w, r, "/edit?ref=refs/heads/"+branch, http.StatusTemporaryRedirect)
+						http.Redirect(w, r, "/edit?ref=refs/heads/"+branch, http.StatusSeeOther)
 						return ErrHandled
 					}
 				}

cmd/gobookmarks/serve.go

@@ -641,7 +641,7 @@ func runTemplate(tmpl string) func(http.ResponseWriter, *http.Request) {
 
 func redirectToHandler(toUrl string) func(http.ResponseWriter, *http.Request) {
 	return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		http.Redirect(w, r, toUrl, http.StatusTemporaryRedirect)
+		http.Redirect(w, r, toUrl, http.StatusSeeOther)
 	})
 }
 
@@ -668,7 +668,7 @@ func redirectToHandlerBranchToRef(toUrl string) func(http.ResponseWriter, *http.
 			qs.Set("edit", edit)
 		}
 		u.RawQuery = qs.Encode()
-		http.Redirect(w, r, u.String(), http.StatusTemporaryRedirect)
+		http.Redirect(w, r, u.String(), http.StatusSeeOther)
 	})
 }
 
@@ -684,7 +684,7 @@ func redirectToHandlerTabPage(toUrl string) func(http.ResponseWriter, *http.Requ
 			qs.Set("edit", edit)
 		}
 		u.RawQuery = qs.Encode()
-		http.Redirect(w, r, u.String(), http.StatusTemporaryRedirect)
+		http.Redirect(w, r, u.String(), http.StatusSeeOther)
 	})
 }