fix(db): revert statement_timeout startup options breaking pooled connections (#4284)

This commit is contained in:
Waleed
2026-04-23 23:31:52 -07:00
committed by GitHub
parent 91ccbb9921
commit ccb5f1e690
2 changed files with 0 additions and 22 deletions

View File

@@ -23,10 +23,6 @@ import { env } from '@/env'
const logger = createLogger('SocketDatabase')
const connectionString = env.DATABASE_URL
/**
* Server-side safety net for runaway queries and abandoned transactions.
* See `packages/db/index.ts` for rationale.
*/
const socketDb = drizzle(
postgres(connectionString, {
prepare: false,
@@ -34,9 +30,6 @@ const socketDb = drizzle(
connect_timeout: 20,
max: 30,
onnotice: () => {},
connection: {
options: '-c statement_timeout=90000 -c idle_in_transaction_session_timeout=90000',
},
}),
{ schema }
)

View File

@@ -10,27 +10,12 @@ if (!connectionString) {
throw new Error('Missing DATABASE_URL environment variable')
}
/**
* Server-side safety net for runaway queries and abandoned transactions:
* - `statement_timeout=90000` kills any single statement still running
* after 90s. Protects against pathological queries.
* - `idle_in_transaction_session_timeout=90000` kills a session that has
* opened a transaction and gone idle for 90s. Protects against
* transactions that hold row locks while waiting on external I/O.
*
* These are last-resort caps — application code should never approach
* them. Migrations or admin scripts that legitimately need longer limits
* must construct their own client with overrides.
*/
const postgresClient = postgres(connectionString, {
prepare: false,
idle_timeout: 20,
connect_timeout: 30,
max: 30,
onnotice: () => {},
connection: {
options: '-c statement_timeout=90000 -c idle_in_transaction_session_timeout=90000',
},
})
export const db = drizzle(postgresClient, { schema })