fix: change event handlers registration style (#474)

graffle-js · Mar 11, 2023 · 8a98925 · 8a98925
1 parent 6ba1648
commit 8a98925
Show file tree

Hide file tree

Showing 4 changed files with 143 additions and 139 deletions.
diff --git a/package.json b/package.json
@@ -69,6 +69,7 @@
     "graphql": "14 - 16"
   },
   "devDependencies": {
+    "@graphql-tools/schema": "^9.0.16",
     "@prisma-labs/prettier-config": "^0.1.0",
     "@tsconfig/node16": "^1.0.3",
     "@types/body-parser": "^1.19.2",

diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/src/graphql-ws.ts b/src/graphql-ws.ts
@@ -93,22 +93,22 @@ export class GraphQLWebSocketClient {
   constructor(socket: WebSocket, { onInit, onAcknowledged, onPing, onPong }: SocketHandler) {
     this.socket = socket
 
-    socket.onopen = async (e) => {
+    socket.addEventListener('open', async (e) => {
       this.socketState.acknowledged = false
       this.socketState.subscriptions = {}
       socket.send(ConnectionInit(onInit ? await onInit() : null).text)
-    }
+    })
 
-    socket.onclose = (e) => {
+    socket.addEventListener('close', (e) => {
       this.socketState.acknowledged = false
       this.socketState.subscriptions = {}
-    }
+    })
 
-    socket.onerror = (e) => {
+    socket.addEventListener('error', (e) => {
       console.error(e)
-    }
+    })
 
-    socket.onmessage = (e) => {
+    socket.addEventListener('message', (e) => {
       try {
         const message = parseMessage(e.data)
         switch (message.type) {
@@ -176,7 +176,7 @@ export class GraphQLWebSocketClient {
         socket.close(1006)
       }
       socket.close(4400, `Unknown graphql-ws message.`)
-    }
+    })
   }
 
   private makeSubscribe<T, V extends Variables, E>(