fix(ws): error message

Author: Daniele Briggi

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@sqlitecloud/drivers",
-  "version": "1.0.653",
+  "version": "1.0.715",
   "description": "SQLiteCloud drivers for Typescript/Javascript in edge, web and node clients",
   "main": "./lib/index.js",
   "types": "./lib/index.d.ts",

package.json

@@ -44,10 +44,11 @@ export class SQLiteCloudWebsocketConnection extends SQLiteCloudConnection {
 
         this.socket.on('connect_error', (error: any) => {
           this.close()
-          callback?.call(
-            this,
-            new SQLiteCloudError(JSON.parse(error.context.responseText).message || 'Connection error', { errorCode: 'ERR_CONNECTION_ERROR' })
-          )
+          let message = error.message || 'Connection error'
+          if (typeof error.context == 'object' && error.context.responseText) {
+            message = typeof error.context.responseText == 'string' ? error.context.responseText : JSON.parse(error.context.responseText)?.message
+          }
+          callback?.call(this, new SQLiteCloudError(message, { errorCode: 'ERR_CONNECTION_ERROR' }))
         })
 
         this.socket.on('error', (error: Error) => {

src/drivers/connection-ws.ts

@@ -6,13 +6,7 @@ import { SQLiteCloudConnection } from '../src/drivers/connection'
 import { SQLiteCloudWebsocketConnection } from '../src/drivers/connection-ws'
 import { SQLiteCloudCommand } from '../src/drivers/types'
 import { SQLiteCloudError } from '../src/index'
-import {
-  EXPECT_SPEED_MS,
-  getChinookConfig,
-  getChinookWebsocketConnection,
-  LONG_TIMEOUT,
-  WARN_SPEED_MS
-} from './shared'
+import { EXPECT_SPEED_MS, getChinookConfig, getChinookWebsocketConnection, LONG_TIMEOUT, WARN_SPEED_MS } from './shared'
 
 describe('connection-ws', () => {
   let chinook: SQLiteCloudConnection
@@ -78,6 +72,27 @@ describe('connection-ws', () => {
     })
   })
 */
+
+    it(
+      'should catch connect_error on connection errors',
+      done => {
+        const configObj = getChinookConfig()
+        configObj.usewebsocket = true
+        configObj.host = 'non.existing.host.name'
+        let connection: SQLiteCloudWebsocketConnection | null = null
+        connection = new SQLiteCloudWebsocketConnection(configObj, error => {
+          try {
+            expect(error).toBeDefined()
+            expect(error?.message.startsWith('Error: getaddrinfo ENOTFOUND non.existing.host.name')).toBe(true)
+            connection?.close()
+          } finally {
+            done()
+          }
+        })
+      },
+      LONG_TIMEOUT
+    )
+
     describe('send test commands', () => {
       it('should test integer', done => {
         chinook.sendCommands('TEST INTEGER', (error, results) => {