OpenNeo/impress-2020
OpenNeo/impress-2020
1
0
Fork 0
Code Pull requests Activity

Oops, fix db pooling for scripts

Browse source 
Right, ok, `db.close()` needs to be `db.end()` now.

This probably didn't break the user-syncing cron job though, because that doesn't automatically update I think? so it should still be comfortably running older version of the code that should still work just fine
This commit is contained in:
Emi Matchu 2022-02-03 16:14:40 -08:00
parent 3744a476e5
commit c1eef6222b
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
scripts/cache-asset-manifests.js
View file

@ -52,7 +52,7 @@ async function cacheAssetManifests(db) {
// Make it a JSON string, then escape the string for the query. // Make it a JSON string, then escape the string for the query.
// Hacky for sure! // Hacky for sure!
const escapedManifest = JSON.stringify(JSON.stringify(manifest)); const escapedManifest = JSON.stringify(JSON.stringify(manifest));
console.log( console.info(
`UPDATE swf_assets SET manifest = ${escapedManifest}, ` + `UPDATE swf_assets SET manifest = ${escapedManifest}, ` +
`manifest_cached_at = CURRENT_TIMESTAMP() ` + `manifest_cached_at = CURRENT_TIMESTAMP() ` +
`WHERE id = ${row.id} LIMIT 1;` `WHERE id = ${row.id} LIMIT 1;`
@ -115,10 +115,10 @@ async function main() {
try { try {
await cacheAssetManifests(db); await cacheAssetManifests(db);
} catch (e) { } catch (e) {
db.close(); db.end();
throw e; throw e;
} }
db.close(); db.end();
} }
main().catch((e) => { main().catch((e) => {

2
scripts/export-users-to-auth0.js
View file

@ -71,7 +71,7 @@ async function main() {
); );
users = rows.map(normalizeRow); users = rows.map(normalizeRow);
} finally { } finally {
db.close(); db.end();
} }
let i = 0; let i = 0;