Make all default return of photo URLs to be empty string

timwangmusic · Oct 29, 2023 · 224c48b · 224c48b
1 parent 3eb3b61
commit 224c48b
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/iowrappers/photos_client.go b/iowrappers/photos_client.go
@@ -46,16 +46,17 @@ func (photoClient *PhotoHttpClient) GetPhotoURL(photoRef string) PhotoURL {
 	res, err := photoClient.client.Get(reqURL)
 	if err != nil {
 		Logger.Fatal(err)
-		return photoURL
+		return ""
 	}
 	body, err := io.ReadAll(res.Body)
 	res.Body.Close()
 	if res.StatusCode != 302 {
 		Logger.Warnf("status code should be 302, but is %d", res.StatusCode)
+		return ""
 	}
 	if err != nil {
 		Logger.Fatal(err)
-		return photoURL
+		return ""
 	}
 
 	photoURL, err = parseHTML(body, isHtmlAnchor, isValidPhotoUrl)