Merge pull request #556 from witheve/fix-ping

all messages to the server should be an object

Author: ibdknox

src/client.ts

@@ -214,7 +214,7 @@ export class EveClient {
     // ping the server so that the connection isn't overzealously
     // closed
     setInterval(() => {
-      this.socketSend("\"PING\"");
+      this.socketSend(JSON.stringify({type: "ping"}));
     }, 30000);
   }
 

src/runtime/runtimeClient.ts

@@ -97,7 +97,14 @@ export abstract class RuntimeClient {
 
   handleEvent(json:string) {
     let data = JSON.parse(json);
-    if(typeof data !== "object") throw new Error(data);
+
+    // Events are expected to be objects that have a type property
+    // if they aren't, we toss the event out
+    if(typeof data !== "object" || data.type === undefined) {
+      console.error("Got invalid JSON event: " + json);
+      return;
+    }
+
     if(data.type === "event") {
       if(!this.evaluation) return;
       // console.info("EVENT", json);