Merge pull request #16 from OperationSpark/chore/redirect-status

Switch redirect status from permanent to temp
OperationSpark · Jan 16, 2023 · 645a981 · 645a981
2 parents c7bc897 + 6e0e75e
commit 645a981
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/handlers/handlers.go b/handlers/handlers.go
@@ -182,7 +182,7 @@ func (s *ShortyService) ServeResolver(w http.ResponseWriter, r *http.Request) {
 		// Redirect even if there is an error. Client should not suffer if the clicks can't be updated.
 		fmt.Fprintf(os.Stderr, "could not update TotalClick count: %v", err)
 	}
-	http.Redirect(w, r, link.OriginalUrl, http.StatusPermanentRedirect)
+	http.Redirect(w, r, link.OriginalUrl, http.StatusTemporaryRedirect)
 }
 
 func (s *ShortyService) createLink(w http.ResponseWriter, r *http.Request) {

diff --git a/integration_test.go b/integration_test.go
@@ -396,7 +396,7 @@ func TestCreateLinkAndRedirect(t *testing.T) {
 
 		server.ServeHTTP(redirectResp, useLinkReq)
 
-		testutil.AssertStatus(t, redirectResp.Code, http.StatusPermanentRedirect)
+		testutil.AssertStatus(t, redirectResp.Code, http.StatusTemporaryRedirect)
 		testutil.AssertContains(t, redirectResp.Body.String(), originalURL)
 
 		// Check click count increment