Skip to content

Commit 43e7fcf

Browse files
committed
Fix DB health check to handle pg results correctly
1 parent 359bd0e commit 43e7fcf

1 file changed

Lines changed: 18 additions & 3 deletions

File tree

server/db.js

Lines changed: 18 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,18 +33,33 @@ pool.on('error', (err) =>
3333
// It also logs query duration in non‑production environments to help
3434
// track slow queries during development.
3535

36+
// export async function query(sql, params = []) {
37+
// const start = Date.now();
38+
// const res = await pool.query(sql, params);
39+
// const ms = Date.now() - start;
40+
41+
// if (process.env.NODE_ENV !== 'production') {
42+
// console.log(`SQL ${ms}ms: `, sql, params);
43+
// }
44+
// return res;
45+
// }
46+
47+
// export async function healthCheck() {
48+
// const rows = await query('select 1 as ok');
49+
// return rows?.[0]?.ok === 1;
50+
// }
3651
export async function query(sql, params = []) {
3752
const start = Date.now();
3853
const res = await pool.query(sql, params);
3954
const ms = Date.now() - start;
40-
4155
if (process.env.NODE_ENV !== 'production') {
4256
console.log(`SQL ${ms}ms: `, sql, params);
4357
}
4458
return res;
4559
}
4660

4761
export async function healthCheck() {
48-
const rows = await query('select 1 as ok');
49-
return rows?.[0]?.ok === 1;
62+
const { rows } = await query('select 1 as ok');
63+
const ok = rows?.[0]?.ok;
64+
return ok === 1 || ok === '1';
5065
}

0 commit comments

Comments
 (0)