Update MySQL handler to add timeout for DNS query failures

Signed-off-by: Michael Irwin <[email protected]> I was helping troubleshoot an issue with someone, who ended up having a typo in their compose file where the MySQL service was using the name "msyql", which obviously causes DNS to fail. But, since the db init doesn't time out, it just looks hung. This fixes that.
chanthornyoeun · Dec 13, 2022 · 8dff10b · 8dff10b
1 parent b4d6426
commit 8dff10b
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/app/src/persistence/mysql.js b/app/src/persistence/mysql.js
@@ -21,7 +21,12 @@ async function init() {
     const password = PASSWORD_FILE ? fs.readFileSync(PASSWORD_FILE) : PASSWORD;
     const database = DB_FILE ? fs.readFileSync(DB_FILE) : DB;
 
-    await waitPort({ host, port : 3306});
+    await waitPort({ 
+        host, 
+        port: 3306,
+        timeout: 10000,
+        waitForDns: true,
+    });
 
     pool = mysql.createPool({
         connectionLimit: 5,