fix: double decoding of json (#319)

opentok · Jan 29, 2024 · ee9eb27 · ee9eb27
1 parent 7a2a65e
commit ee9eb27
Show file tree

Hide file tree

Showing 3 changed files with 953 additions and 958 deletions.
diff --git a/lib/client.js b/lib/client.js
@@ -70,7 +70,8 @@ const api = ({
       const bodyText = await response.text();
       let body = bodyText;
 
-      switch (response.headers.get('content-type')) {
+      const [contentType] = (response.headers.get('content-type') || '').split(';');
+      switch (contentType) {
         case 'application/x-www-form-urlencoded':
           body = response.body
             ? new URLSearchParams(body)
@@ -312,9 +313,7 @@ Client.prototype.listBroadcasts = function listBroadcasts(queryString, cb) {
     url:url,
     method: 'GET',
     headers: this.generateHeaders(),
-    callback: (err, response ) => {
-      const items = response ? JSON.parse(response) : response;
-
+    callback: (err, items) => {
       cb(
         err,
         items?.items.map((item) => new Broadcast(Client, JSON.stringify(item))),
@@ -410,8 +409,7 @@ Client.prototype.listStreams = function listStreams(sessionId, cb) {
     url: url,
     method: 'GET',
     headers: this.generateHeaders(),
-    callback: (err, response) => {
-      const body = response ? JSON.parse(response) : response;
+    callback: (err, body) => {
       cb(err, body?.items.map((stream) => new Stream(JSON.stringify(stream))))
     },
   });