Fix redirect after login

The redirect caused the browser to be redirected to '/', but they were issueing POST requests instead of GET.
vshn · Nov 10, 2021 · 2ea4d9d · 2ea4d9d
1 parent 90308e9
commit 2ea4d9d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/pkg/web/session_handlers.go b/pkg/web/session_handlers.go
@@ -58,7 +58,7 @@ func (s Server) Login() http.Handler {
 			HttpOnly: true,
 			Secure:   true,
 		})
-		http.Redirect(w, r, "/", http.StatusTemporaryRedirect)
+		http.Redirect(w, r, "/", http.StatusFound)
 	})
 }
 

diff --git a/test/session_test.go b/test/session_test.go
@@ -85,8 +85,8 @@ func TestLoginSuccess(t *testing.T) {
 	res := httptest.NewRecorder()
 	newServer(odooMock.URL).ServeHTTP(res, req)
 
-	is.Equal(res.Code, http.StatusTemporaryRedirect) // HTTP status
-	is.Equal(res.Header().Get("Location"), "/")      // Location header
+	is.Equal(res.Code, http.StatusFound)        // HTTP status
+	is.Equal(res.Header().Get("Location"), "/") // Location header
 
 	is.Equal(1, len(res.Result().Cookies())) // number of cookies
 	c := res.Result().Cookies()[0]