don't copy response for og:image if status not success

Southclaws · Dec 10, 2023 · fa32eb0 · fa32eb0 · vercel · Dec 10, 2023
1 parent 0670dc0
commit fa32eb0
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/app/services/hydrator/fetcher/fetcher.go b/app/services/hydrator/fetcher/fetcher.go
@@ -106,6 +106,11 @@ func (s *service) copy(ctx context.Context, url string) (*asset.Asset, error) {
 		return nil, fault.Wrap(err, fctx.With(ctx))
 	}
 
+	if resp.StatusCode != http.StatusOK {
+		ctx = fctx.WithMeta(ctx, "status", resp.Status)
+		return nil, fault.Wrap(fault.New("failed to get"), fctx.With(ctx))
+	}
+
 	a, err := s.as.Upload(ctx, resp.Body, resp.ContentLength)
 	if err != nil {
 		return nil, fault.Wrap(err, fctx.With(ctx))