Skip to content

Fix database connection leak when deleting repos via web UI: Fixes #3392#3400

Merged
sgoggins merged 1 commit intoaugurlabs:mainfrom
shlokgilda:feature/issue-3392-too-many-clients
Nov 14, 2025
Merged

Fix database connection leak when deleting repos via web UI: Fixes #3392#3400
sgoggins merged 1 commit intoaugurlabs:mainfrom
shlokgilda:feature/issue-3392-too-many-clients

Conversation

@shlokgilda
Copy link
Copy Markdown
Collaborator

@shlokgilda shlokgilda commented Nov 12, 2025

Description

  • Fixed database connection leak that occurred when deleting multiple repos through the web UI. Fixes FATAL: sorry, too many clients already when deleting lots of repos via the web UI #3392.
  • The delete operation was creating new database connections without properly closing them, eventually hitting PostgreSQL's connection limit and causing "FATAL: sorry, too many clients already" errors
  • Ensured proper session/connection management in the repo deletion flow

Notes for Reviewers

  • Should no longer encounter "too many clients already" errors even when deleting many repos

Signed commits

  • Yes, I signed my commits.

Copy link
Copy Markdown
Collaborator

@sgoggins sgoggins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this should fix connection leaks.

@sgoggins sgoggins added API Related to Augur's metrics API bug-fix Fixes a bug ready Items tested and seeking additional approvals or a merge. Usually for items under active development labels Nov 13, 2025
@sgoggins sgoggins self-assigned this Nov 13, 2025
Signed-off-by: Shlok Gilda <gildashlok@hotmail.com>
@shlokgilda shlokgilda force-pushed the feature/issue-3392-too-many-clients branch from 679f6c5 to 15b2dcc Compare November 13, 2025 19:40
@sgoggins sgoggins merged commit 74da538 into augurlabs:main Nov 14, 2025
10 checks passed
@MoralCode
Copy link
Copy Markdown
Collaborator

@sgoggins were you able to test and confirm that this fixes the issue?

@shlokgilda shlokgilda deleted the feature/issue-3392-too-many-clients branch November 17, 2025 01:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API Related to Augur's metrics API bug-fix Fixes a bug ready Items tested and seeking additional approvals or a merge. Usually for items under active development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FATAL: sorry, too many clients already when deleting lots of repos via the web UI

3 participants