Jelajahi Sumber

Do not log website without icon as warning

Frédéric Guillot 2 tahun lalu
induk
melakukan
5ac3489ee5
2 mengubah file dengan 4 tambahan dan 4 penghapusan
  1. 1 1
      internal/reader/handler/handler.go
  2. 3 3
      internal/reader/icon/finder.go

+ 1 - 1
internal/reader/handler/handler.go

@@ -243,7 +243,7 @@ func checkFeedIcon(store *storage.Storage, feedID int64, websiteURL, feedIconURL
 	if !store.HasIcon(feedID) {
 		iconFinder := icon.NewIconFinder(websiteURL, feedIconURL, userAgent, fetchViaProxy, allowSelfSignedCertificates)
 		if icon, err := iconFinder.FindIcon(); err != nil {
-			slog.Warn("Unable to find feed icon",
+			slog.Debug("Unable to find feed icon",
 				slog.Int64("feed_id", feedID),
 				slog.String("website_url", websiteURL),
 				slog.String("feed_icon_url", feedIconURL),

+ 3 - 3
internal/reader/icon/finder.go

@@ -189,16 +189,16 @@ func (f *IconFinder) DownloadIcon(iconURL string) (*model.Icon, error) {
 
 	response, err := clt.Get()
 	if err != nil {
-		return nil, fmt.Errorf("icon: unable to download iconURL: %v", err)
+		return nil, fmt.Errorf("icon: unable to download icon %s: %v", iconURL, err)
 	}
 
 	if response.HasServerFailure() {
-		return nil, fmt.Errorf("icon: unable to download icon: status=%d", response.StatusCode)
+		return nil, fmt.Errorf("icon: unable to download icon %s: status=%d", iconURL, response.StatusCode)
 	}
 
 	body, err := io.ReadAll(response.Body)
 	if err != nil {
-		return nil, fmt.Errorf("icon: unable to read downloaded icon: %v", err)
+		return nil, fmt.Errorf("icon: unable to read downloaded icon from %s: %v", iconURL, err)
 	}
 
 	if len(body) == 0 {