Merge pull request #1086 from redlock/master

Fixed crash caused by TypeError due to missing builder.

Author: jaime-ez

CHANGELOG.md

@@ -4,6 +4,12 @@
 
 - Updating dependencies
 
+## [5.1.4] - 2020.10.16
+
+### Fix
+
+- Fixed bug causing the server to crash with older sdks.
+
 ## [5.1.3] - 2020.08.08
 
 ### Fix

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@deepstream/server",
-  "version": "5.1.3",
+  "version": "5.1.4",
   "description": "a scalable server for realtime webapps",
   "main": "./dist/src/deepstream.io.js",
   "bin": {

package.json

@@ -213,14 +213,13 @@ const BUILDERS = {
  */
 export const getMessage = (message: Message, isAck: boolean = false): string => {
   if (!BUILDERS[message.topic] || !BUILDERS[message.topic][message.action]) {
-    console.log(message, isAck)
+    console.trace('missing builder for', message, isAck)
+    return ''
   }
+  
   const builder = BUILDERS[message.topic][message.action]
-  if (!builder) {
-    console.trace('missing builder for', message)
-    return ''
-  } else {
-    if (
+  
+  if (
       !message.parsedData && !message.data &&
       (
         (message.topic === TOPIC.RPC.BYTE && (message.action === PA.RESPONSE.BYTE || message.action === PA.REQUEST.BYTE)) ||
@@ -238,8 +237,7 @@ export const getMessage = (message: Message, isAck: boolean = false): string =>
       message.data = typed(JSON.parse(message.data.toString()))
     }
 
-    return builder(message, isAck)
-  }
+    return builder(message, isAck)  
 }
 
 /**