Remove unnecessary option copying

okTurtles · Feb 28, 2021 · f875568 · f875568
1 parent bdf4d75
commit f875568
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/backend/utils/pubsub.js b/backend/utils/pubsub.js
@@ -58,8 +58,8 @@ export function createServer (httpServer: Object, options?: Object = {}): Object
     ...options,
     server: httpServer
   })
-  server.customClientHandlers = { ...options.clientHandlers }
-  server.customServerHandlers = { ...options.serverHandlers }
+  server.customClientHandlers = options.clientHandlers || {}
+  server.customServerHandlers = options.serverHandlers || {}
   server.messageHandlers = { ...defaultMessageHandlers, ...options.messageHandlers }
   server.pingInterval = undefined
   server.subscribersByContractID = Object.create(null)

diff --git a/shared/pubsub.js b/shared/pubsub.js
@@ -82,7 +82,7 @@ export const RESPONSE_TYPE = Object.freeze({
  */
 export function createClient (url: string, options?: Object = {}): PubSubClient {
   const client = {
-    customHandlers: { ...options.handlers },
+    customHandlers: options.handlers || {},
     failedAttempts: 0,
     isReconnecting: false,
     listeners: Object.create(null),