safe-global / safe-decoder-service / 27293513379

91%

push

github

Fix connections left idle in transaction exhausting Postgres slots (#297)

- Release the read transaction in get_contract_metadata_task before the
  external metadata download, via a scoped with_db_session_context block, so
  the connection is returned to the pool instead of sitting idle in
  transaction during slow HTTP calls.
- Add with_db_session_context async context manager and reuse it in the
  db_session_context decorator and the startup ABI initialization, ensuring
  the session/connection is always released on exit.
- Set Postgres idle_in_transaction_session_timeout per connection as a
  backstop so leaked transactions are reclaimed (configurable, default 30s).

28 of 34 new or added lines in 5 files covered. (82.35%)

1329 of 1454 relevant lines covered (91.4%)

0.91 hits per line