Avoid re-raising caught exception via future.result() inside error logger

98e65c4c997a82ed8fc782b17ed1511095899117 · 2026-05-18 20:09:15 -0700 · Steve Abrams

Files touched

M fashioncrawler/scraper/depop_scraper.py

Diff

commit 98e65c4c997a82ed8fc782b17ed1511095899117
Author: Steve Abrams <steve@designerwallcoverings.com>
Date:   Mon May 18 20:09:15 2026 -0700

    Avoid re-raising caught exception via future.result() inside error logger
---
 fashioncrawler/scraper/depop_scraper.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fashioncrawler/scraper/depop_scraper.py b/fashioncrawler/scraper/depop_scraper.py
index cdcb42a..c8b3d14 100644
--- a/fashioncrawler/scraper/depop_scraper.py
+++ b/fashioncrawler/scraper/depop_scraper.py
@@ -423,7 +423,7 @@ class DepopScraper(BaseScraper):
                         logger.error("Task canceled:", exc_info=True)
                     except Exception as e:
                         logger.error(
-                            f"Error fetching page source for {future.result()}: {e}",
+                            f"Error fetching page source: {e}",
                             exc_info=True,
                         )
                         future.cancel()

← 17d6b96 Guard main loop against unknown enabled sites instead of cra · back to FashionCrawler · FashionCrawler: realpath containment on do_GET .css path (Ti 387f956 →