Merge pull request #1 from kadirahq/remove-serialization

Remove serialization step because postMessage api does it
storybookjs · Sep 28, 2016 · f0e2545 · f0e2545
2 parents 9f6bb93 + af17685
commit f0e2545
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/index.js b/src/index.js
@@ -25,7 +25,7 @@ export class PostmsgTransport {
         this._buffer.push({ event, resolve, reject });
       });
     }
-    const data = JSON.stringify({ key: this._key, event });
+    const data = { key: this._key, event };
     iframeWindow.postMessage(data, '*');
     return Promise.resolve(null);
   }
@@ -54,7 +54,10 @@ export class PostmsgTransport {
   }
 
   _handleEvent(e) {
-    const { key, event } = JSON.parse(e.data);
+    if(!e.data || typeof(e.data) !== 'object') {
+      return;
+    }
+    const { key, event } = e.data;
     if (key !== this._key) {
       return null;
     }