Add `connection_closed` or `pre_connection_closed` and `post_connection_closed` signals

tomwojcik · January 9, 2025, 10:08am

Currently I’m facing an issue with stale DB connections. It’s probably related to my setup. Having connection_closed signal would simplify my debugging process.

Since 2009, there’s connection_created signal, added in 6f38697f90a14f1450a71c1e40aea0f5df7dca86 (I can’t post the link because the forum complains about An error occurred: Sorry, new users can only put 2 links in a post.…)

I’d like to create a connection_closed signal and connect it here

github.com

django/django/blob/6f38697f90a14f1450a71c1e40aea0f5df7dca86/django/db/backends/base/base.py#L314


      
                      return self.connection.commit()
          
          def _rollback(self):
              if self.connection is not None:
                  with debug_transaction(self, "ROLLBACK"), self.wrap_database_errors:
                      return self.connection.rollback()
          
          def _close(self):
              if self.connection is not None:
                  with self.wrap_database_errors:
                      return self.connection.close()
          
          # ##### Generic wrappers for PEP-249 connection methods #####
          
          @async_unsafe
          def cursor(self):
              """Create a cursor, opening a connection if necessary."""
              return self._cursor()
          
          @async_unsafe
          def commit(self):

This way the connection can be accessed right before closing it.

I’d be happy to implement it. Let me know what you think.

I’m also open to pre_connection_closed and post_connection_closed. Even though not many people will ever use it, I can imagine wanting both in different cases.

initially posted here #36003 (Add `connection_closed` or `pre_connection_closed` and `post_connection_closed` signals) – Django

Topic		Replies	Views
Closing old DB connections outside of the request context Django Internals	1	4380	September 24, 2019
When are db connections closed? Django Internals	16	10443	October 10, 2022
Seemingly random PostgreSQL connection drops while handling Django requests Mystery Errors	21	6406	August 21, 2024
django.db.utils.InterfaceError: connection already closed when updating from django 3.0 to 3.1 Deployment	22	9269	May 21, 2024
Django DB issue (Interface error: connection closed) Mystery Errors	2	1791	November 25, 2023

Add `connection_closed` or `pre_connection_closed` and `post_connection_closed` signals

Related topics