This function is one of the worst for database leaks

[collectoss-issue-migrator]

Note

Migrated from augurlabs/augur#3452
Originally opened by @MoralCode on 2025-12-10

---

https://github.com/chaoss/augur/blob/34f5644e2738c91d2f6b2a4ce81035fa201d9094/augur/tasks/init/__init__.py#L7

every time this is called, a DB connection is opened that never gets closed. This should not be happening.
This has been looked at by several maintainers several times

We need tests and stuff to confirm this

[collectoss-issue-migrator]

Note

Migrated comment from original
Originally posted by @sgoggins on 2025-12-11

---

I think co-pilot found the missing indentation for this problem:

augurlabs/augur@2f1a561

[collectoss-issue-migrator]

Note

Migrated comment from original
Originally posted by @MoralCode on 2025-12-11

---

I really dont think thats the answer

[collectoss-issue-migrator]

Note

Migrated comment from original
Originally posted by @MoralCode on 2025-12-11

---

My plan:

1. Wait for database stuff to be fixed/merged
2. Fix database test fixtures in unit tests
3. Create unit test confirming one connection is leaked every time the function in this issue runs
4. Experiment with fixes until the test passes

[collectoss-issue-migrator]

Note

Migrated comment from original
Originally posted by @SOHAMPAL23 on 2026-01-05

---

Hey @MoralCode can I work on this issue

[collectoss-issue-migrator]

Note

Migrated comment from original
Originally posted by @MoralCode on 2026-01-06

---

This issue is currently on hold until some other changes land first. Its also a very deeply technical change and is not going to be very friendly for a newcomer.

[collectoss-issue-migrator]

Note

Migrated comment from original
Originally posted by @MoralCode on 2026-03-13

---

https://docs.sqlalchemy.org/en/20/orm/session_basics.html

This seems like useful prerequisite understanding to evaluate this issue