Use http 302 for redirection instead of 303
authorr <r@freesoftwareextremist.com>
Tue, 17 Dec 2019 20:15:34 +0000 (20:15 +0000)
committerr <r@freesoftwareextremist.com>
Tue, 17 Dec 2019 20:15:34 +0000 (20:15 +0000)
service/transport.go

index 377ab23e64f81dbfb1db41c03bedabfbcbf993c2..1326c580f99575685094a33272a2083ac23304ae 100644 (file)
@@ -38,7 +38,7 @@ func NewHandler(s Service, staticDir string) http.Handler {
                }
 
                w.Header().Add("Location", location)
-               w.WriteHeader(http.StatusSeeOther)
+               w.WriteHeader(http.StatusFound)
        }).Methods(http.MethodGet)
 
        r.HandleFunc("/signin", func(w http.ResponseWriter, req *http.Request) {
@@ -59,7 +59,7 @@ func NewHandler(s Service, staticDir string) http.Handler {
 
                w.Header().Add("Set-Cookie", fmt.Sprintf("session_id=%s;max-age=%s", sessionId, cookieAge))
                w.Header().Add("Location", url)
-               w.WriteHeader(http.StatusSeeOther)
+               w.WriteHeader(http.StatusFound)
        }).Methods(http.MethodPost)
 
        r.HandleFunc("/oauth_callback", func(w http.ResponseWriter, req *http.Request) {
@@ -72,7 +72,7 @@ func NewHandler(s Service, staticDir string) http.Handler {
                }
 
                w.Header().Add("Location", "/timeline")
-               w.WriteHeader(http.StatusSeeOther)
+               w.WriteHeader(http.StatusFound)
        }).Methods(http.MethodGet)
 
        r.HandleFunc("/timeline", func(w http.ResponseWriter, req *http.Request) {
@@ -110,7 +110,7 @@ func NewHandler(s Service, staticDir string) http.Handler {
                }
 
                w.Header().Add("Location", req.Header.Get("Referer")+"#status-"+id)
-               w.WriteHeader(http.StatusSeeOther)
+               w.WriteHeader(http.StatusFound)
        }).Methods(http.MethodGet)
 
        r.HandleFunc("/unlike/{id}", func(w http.ResponseWriter, req *http.Request) {
@@ -123,7 +123,7 @@ func NewHandler(s Service, staticDir string) http.Handler {
                }
 
                w.Header().Add("Location", req.Header.Get("Referer")+"#status-"+id)
-               w.WriteHeader(http.StatusSeeOther)
+               w.WriteHeader(http.StatusFound)
        }).Methods(http.MethodGet)
 
        r.HandleFunc("/retweet/{id}", func(w http.ResponseWriter, req *http.Request) {
@@ -136,7 +136,7 @@ func NewHandler(s Service, staticDir string) http.Handler {
                }
 
                w.Header().Add("Location", req.Header.Get("Referer")+"#status-"+id)
-               w.WriteHeader(http.StatusSeeOther)
+               w.WriteHeader(http.StatusFound)
        }).Methods(http.MethodGet)
 
        r.HandleFunc("/unretweet/{id}", func(w http.ResponseWriter, req *http.Request) {
@@ -149,7 +149,7 @@ func NewHandler(s Service, staticDir string) http.Handler {
                }
 
                w.Header().Add("Location", req.Header.Get("Referer")+"#status-"+id)
-               w.WriteHeader(http.StatusSeeOther)
+               w.WriteHeader(http.StatusFound)
        }).Methods(http.MethodGet)
 
        r.HandleFunc("/post", func(w http.ResponseWriter, req *http.Request) {
@@ -176,7 +176,7 @@ func NewHandler(s Service, staticDir string) http.Handler {
                        location = "/thread/" + replyToID + "#status-" + id
                }
                w.Header().Add("Location", location)
-               w.WriteHeader(http.StatusSeeOther)
+               w.WriteHeader(http.StatusFound)
        }).Methods(http.MethodPost)
 
        r.HandleFunc("/notifications", func(w http.ResponseWriter, req *http.Request) {
@@ -196,7 +196,7 @@ func NewHandler(s Service, staticDir string) http.Handler {
                // TODO remove session from database
                w.Header().Add("Set-Cookie", fmt.Sprintf("session_id=;max-age=0"))
                w.Header().Add("Location", "/")
-               w.WriteHeader(http.StatusSeeOther)
+               w.WriteHeader(http.StatusFound)
        }).Methods(http.MethodGet)
 
        return r