Merge pull request #942 from GFDRR/always-close-browser

Always close browser
GFDRR · Apr 17, 2024 · eeb8362 · eeb8362
2 parents 3fa143b + 8ad6b85
commit eeb8362
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/thinkhazard/views/pdf.py b/thinkhazard/views/pdf.py
@@ -124,11 +124,12 @@ async def render_page(url):
         logger.info("Got: {}".format(url))
         return BytesIO(await page.pdf({"format": "A4", "printBackground": True}))
 
-    chunks = await asyncio.gather(*[
-        render_page(url) for url in pages
-    ])
-
-    await browser.close()
+    try:
+        chunks = await asyncio.gather(*[
+            render_page(url) for url in pages
+        ])
+    finally:
+        await browser.close()
 
     # merge all pages
     writer = PdfFileWriter()