Merge pull request #17 from tcriess/change-remove-json-double-encoding

Remove json double encoding in ws communication
GRVYDEV · Feb 21, 2021 · 08c2894 · 08c2894
2 parents e937aa9 + dad04cd
commit 08c2894
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/App.js b/src/App.js
@@ -56,7 +56,7 @@ const App = () => {
       socket.send(
         JSON.stringify({
           event: "candidate",
-          data: JSON.stringify(e.candidate),
+          data: e.candidate,
         })
       );
     }
@@ -71,7 +71,7 @@ const App = () => {
         return;
       }
 
-      const offerCandidate = JSON.parse(msg.data);
+      const offerCandidate = msg.data;
 
       if (!offerCandidate) {
         console.log("Failed to parse offer msg data");
@@ -89,7 +89,7 @@ const App = () => {
             socket.send(
               JSON.stringify({
                 event: "answer",
-                data: JSON.stringify(answer),
+                data: answer,
               })
             );
           } catch (e) {
@@ -104,7 +104,7 @@ const App = () => {
         case "info":
           dispatch({
             type: "info",
-            viewers: JSON.parse(msg.data).no_connections,
+            viewers: msg.data.no_connections,
           });
       }
     };