Fixing profile image tag (#7435)

* Fixing profile image tag

* Fixing unit test

* Fixing unit test

api4/user.go

@@ -214,10 +214,10 @@ func getProfileImage(c *Context, w http.ResponseWriter, r *http.Request) {
 			w.Header().Set("Cache-Control", fmt.Sprintf("max-age=%v, public", 5*60)) // 5 mins
 		} else {
 			w.Header().Set("Cache-Control", fmt.Sprintf("max-age=%v, public", 24*60*60)) // 24 hrs
+			w.Header().Set(model.HEADER_ETAG_SERVER, etag)
 		}
 
 		w.Header().Set("Content-Type", "image/png")
-		w.Header().Set(model.HEADER_ETAG_SERVER, etag)
 		w.Write(img)
 	}
 }

api4/user_test.go

@@ -866,8 +866,8 @@ func TestGetProfileImage(t *testing.T) {
 	}
 
 	_, resp = Client.GetProfileImage(user.Id, resp.Etag)
-	if resp.StatusCode != http.StatusNotModified {
-		t.Fatal("Should have hit etag")
+	if resp.StatusCode == http.StatusNotModified {
+		t.Fatal("Shouldn't have hit etag")
 	}
 
 	_, resp = Client.GetProfileImage("junk", "")