From cef69c7db92f63198a8e4d29acd4607e7fdea70e Mon Sep 17 00:00:00 2001 From: Christopher Patti Date: Mon, 2 Feb 2026 15:51:51 -0500 Subject: [PATCH 1/2] fix: Remove pgbouncer related configuration mitodl/hq#9395 - Don't disable server side cursors --- main/settings.py | 4 ---- main/settings_test.py | 8 -------- 2 files changed, 12 deletions(-) diff --git a/main/settings.py b/main/settings.py index f9eeb66c33..ca7bba8f9f 100644 --- a/main/settings.py +++ b/main/settings.py @@ -307,10 +307,6 @@ "sqlite:///{}".format(os.path.join(BASE_DIR, "db.sqlite3")), # noqa: PTH118 ) ) -DEFAULT_DATABASE_CONFIG["DISABLE_SERVER_SIDE_CURSORS"] = get_bool( - "MITOL_DB_DISABLE_SS_CURSORS", - True, # noqa: FBT003 -) DEFAULT_DATABASE_CONFIG["CONN_MAX_AGE"] = get_int("MITOL_DB_CONN_MAX_AGE", 0) if get_bool("MITOL_DB_DISABLE_SSL", False): # noqa: FBT003 diff --git a/main/settings_test.py b/main/settings_test.py index 3d4b57b6eb..5212a875d5 100644 --- a/main/settings_test.py +++ b/main/settings_test.py @@ -164,14 +164,6 @@ def test_required_settings(self): ): self.reload_settings() - def test_server_side_cursors_disabled(self): - """DISABLE_SERVER_SIDE_CURSORS should be true by default""" - with mock.patch.dict("os.environ", REQUIRED_SETTINGS): - settings_vars = self.reload_settings() - assert ( - settings_vars["DEFAULT_DATABASE_CONFIG"]["DISABLE_SERVER_SIDE_CURSORS"] - is True - ) def test_server_side_cursors_enabled(self): """DISABLE_SERVER_SIDE_CURSORS should be false if MITOL_DB_DISABLE_SS_CURSORS is false""" From b0811d9060f7b42cd3a7a410acd2ecb14a84ab53 Mon Sep 17 00:00:00 2001 From: "pre-commit-ci[bot]" <66853113+pre-commit-ci[bot]@users.noreply.github.com> Date: Mon, 2 Feb 2026 21:08:01 +0000 Subject: [PATCH 2/2] [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --- main/settings_test.py | 1 - 1 file changed, 1 deletion(-) diff --git a/main/settings_test.py b/main/settings_test.py index 5212a875d5..fe2e7b0360 100644 --- a/main/settings_test.py +++ b/main/settings_test.py @@ -164,7 +164,6 @@ def test_required_settings(self): ): self.reload_settings() - def test_server_side_cursors_enabled(self): """DISABLE_SERVER_SIDE_CURSORS should be false if MITOL_DB_DISABLE_SS_CURSORS is false""" with mock.patch.dict(