Skip to content
This repository was archived by the owner on Apr 26, 2026. It is now read-only.

feature: add new chains & fix: connection pool leak #30

Merged
nimrod-teich merged 7 commits intomainnetfrom
testnet
Apr 7, 2026
Merged

feature: add new chains & fix: connection pool leak #30
nimrod-teich merged 7 commits intomainnetfrom
testnet

Conversation

@ssejzer
Copy link
Copy Markdown

@ssejzer ssejzer commented Apr 6, 2026

  • Connection pool leak: The finally block in db.ts:308 had an inverted condition (if (this.connections.includes(conn)) return;) that skipped connection cleanup, leaking every DB connection until the pool filled to
    1000/1000. On cache miss, endpoints like /listProviders would hang forever waiting for a free connection. Fixed by adding the missing !.
  • Broken .client.end() call: The leak fix exposed a second latent bug — (conn.db as any).client.end() threw TypeError because the drizzle ORM wrapper doesn't expose the raw postgres client at .client. This never
    crashed before because the inverted condition prevented the line from ever executing. Fixed by storing the raw postgres() sql client as sqlClient on the DbConnection interface and calling conn.sqlClient.end()
    directly.

nimrod-teich and others added 5 commits March 24, 2026 14:07
Add XLM, SONIC, UNION, BTC, BCH, AVAX and their testnet variants.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add icons for XLM, XLMT, Sonic, Union, BTC testnet, BCH, Avalanche testnet,
and sync missing entries from jsinfo-ui (polygona, bnb, bera, hyperliquid).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…o-ui

Add XLM, XLMT, SONIC, BTC, BTCT, BCH, BCHT and sync BERAT2, BERA, BNB,
BNT, POLYGONA, MOVEMENTT, HYPERLIQUID, HYPERLIQUIDT.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@ssejzer ssejzer marked this pull request as draft April 6, 2026 12:56
Sebastian Sejzer 🔥 and others added 2 commits April 6, 2026 16:36
@ssejzer ssejzer temporarily deployed to Configure CI/CD April 6, 2026 13:51 — with GitHub Actions Inactive
@ssejzer ssejzer marked this pull request as ready for review April 7, 2026 05:16
@nimrod-teich nimrod-teich merged commit beb7df5 into mainnet Apr 7, 2026
1 check passed
@NadavLevi NadavLevi deleted the testnet branch April 7, 2026 08:14
@NadavLevi NadavLevi restored the testnet branch April 7, 2026 08:15
@NadavLevi NadavLevi temporarily deployed to Configure CI/CD April 7, 2026 08:15 — with GitHub Actions Inactive
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants