Allow period in listen channel names - fix #495

porsager · Oct 5, 2022 · a12108a · a12108a
1 parent a0fde1f
commit a12108a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/index.js b/src/index.js
@@ -176,7 +176,9 @@ function Postgres(a, b) {
       return { state: result.state, unlisten }
     }
 
-    channels[name] = { result: sql`listen ${ sql(name) }`, listeners: [listener] }
+    channels[name] = { result: sql`listen ${
+      sql.unsafe('"' + name.replace(/"/g, '""') + '"')
+    }`, listeners: [listener] }
     const result = await channels[name].result
     listener.onlisten && listener.onlisten()
     return { state: result.state, unlisten }

diff --git a/tests/index.js b/tests/index.js
@@ -700,7 +700,7 @@ t('multiple listeners work after a reconnect', async() => {
 
 t('listen and notify with weird name', async() => {
   const sql = postgres(options)
-  const channel = 'wat-;ø§'
+  const channel = 'wat-;.ø.§'
   const result = await new Promise(async r => {
     await sql.listen(channel, r)
     sql.notify(channel, 'works')