Merge pull request #14 from observablehq/visnup/snowflake-already-connecting

Fix Snowflake already connecting bug

Author: visnup

lib/snowflake.js

@@ -5,11 +5,11 @@ import snowflake from "snowflake-sdk";
 
 export default url => {
   url = new URL(url);
-  const {host: account, username, password, pathname, searchParams} = new URL(
+  const {host, username, password, pathname, searchParams} = new URL(
     url
   );
   const connection = snowflake.createConnection({
-    account,
+    account: host,
     username,
     password,
     database: pathname.slice(1),
@@ -18,17 +18,20 @@ export default url => {
     role: searchParams.get("role")
   });
 
+  const connecting = new WeakSet();
   return async function query(req, res) {
     const body = await json(req);
     const {sql, params} = body;
 
     const client = await new Promise((resolve, reject) => {
-      if (connection.isUp()) return resolve(connection);
+      if (connection.isUp() || connecting.has(connection))
+        return resolve(connection);
       snowflake.configure({ocspFailOpen: false});
       connection.connect((err, conn) => {
         if (err) reject(err);
         else resolve(conn);
       });
+      connecting.add(connection);
     });
 
     const statement = client.execute({sqlText: sql, binds: params});