let's exception not bubble

ensure we can catch correctly exceptions based on BaseException. Note: patch was origninally proposed by the pr benoitc#2923, but original author closed it. Fix benoitc#2923
pajod · Dec 7, 2023 · 4023228 · 4023228
1 parent ca9162d
commit 4023228
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/gunicorn/workers/base_async.py b/gunicorn/workers/base_async.py
@@ -82,7 +82,7 @@ def handle(self, listener, client, addr):
                     self.log.debug("Ignoring socket not connected")
                 else:
                     self.log.debug("Ignoring EPIPE")
-        except Exception as e:
+        except BaseException as e:
             self.handle_error(req, client, addr, e)
         finally:
             util.close(client)

diff --git a/gunicorn/workers/sync.py b/gunicorn/workers/sync.py
@@ -154,7 +154,7 @@ def handle(self, listener, client, addr):
                     self.log.debug("Ignoring socket not connected")
                 else:
                     self.log.debug("Ignoring EPIPE")
-        except Exception as e:
+        except BaseException as e:
             self.handle_error(req, client, addr, e)
         finally:
             util.close(client)