Revert "Redirect to bouncer with 307 (#92)" (#94)

This reverts commit 7828dfd.

Author: oremj

stubservice/stubhandlers/stubhandler_test.go

@@ -242,7 +242,7 @@ func TestStubServiceErrorCases(t *testing.T) {
 		recorder := fetchURL(`http://test/?product=firefox-stub&os=win&lang=en-US`)
 		code := recorder.Code
 		location := recorder.HeaderMap.Get("Location")
-		if code != 307 || location != "https://download.mozilla.org/?lang=en-US&os=win&product=firefox-stub" {
+		if code != 302 || location != "https://download.mozilla.org/?lang=en-US&os=win&product=firefox-stub" {
 			t.Errorf("service did not return bouncer redirect status: %d loc: %s", code, location)
 		}
 	})
@@ -251,7 +251,7 @@ func TestStubServiceErrorCases(t *testing.T) {
 		recorder := fetchURL(`http://test/?product=firefox-stub&os=win&lang=en-US&attribution_code=invalidcode`)
 		code := recorder.Code
 		location := recorder.HeaderMap.Get("Location")
-		if code != 307 || location != "https://download.mozilla.org/?lang=en-US&os=win&product=firefox-stub" {
+		if code != 302 || location != "https://download.mozilla.org/?lang=en-US&os=win&product=firefox-stub" {
 			t.Errorf("service did not return bouncer redirect status: %d loc: %s", code, location)
 		}
 	})

stubservice/stubhandlers/stubservice.go

@@ -30,7 +30,7 @@ func (s *stubService) ServeHTTP(w http.ResponseWriter, req *http.Request) {
 
 	redirectBouncer := func() {
 		backupURL := bouncerURL(query.Get("product"), query.Get("lang"), query.Get("os"))
-		http.Redirect(w, req, backupURL, http.StatusTemporaryRedirect)
+		http.Redirect(w, req, backupURL, http.StatusFound)
 	}
 
 	attributionCode := query.Get("attribution_code")