Merge pull request #831 from Gluejar/more_online

add exception handling in stapler
2019-03-04 19:48:16 -05:00 · 2019-03-04 19:48:16 -05:00 · e618e5da8d
parent 11090aa0d5 d87578c5a0
commit e618e5da8d
1 changed files with 5 additions and 1 deletions
--- a/core/pdf.py
+++ b/core/pdf.py
@ -66,7 +66,11 @@ def test_pdf(pdf_file):
 def staple_pdf(urllist, user_agent=settings.USER_AGENT):
    merger = PdfFileMerger(strict=False)
    for url in urllist:
-        response = requests.get(url, headers={"User-Agent": user_agent})
+        try:
+            response = requests.get(url, headers={"User-Agent": user_agent})
+        except requests.exceptions.ConnectionError:
+            logger.error("Error getting url: %s", url)
+            return None
        if response.status_code == 200:
            try:
                merger.append(BytesIO(response.content))