Merge pull request #8 from aduyng/fix-204-JSON-parsing-issue

fixed the 204 JSON parsing issue
lpg2709 · Jun 12, 2023 · 8334c0f · 8334c0f
2 parents 75e8605 + b8187a8
commit 8334c0f
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/src/utils/requests.js b/src/utils/requests.js
@@ -138,13 +138,21 @@ module.exports = {
 				if (res.statusCode != 204) {
 					reject(res.statusCode);
 				}
-
+				// 204 should not return any data, but if it does, we'll parse it
 				res.on('data', function (chunk) {
 					str += chunk;
 				});
 
 				res.on('end', function () {
-					resolve(JSON.parse(str));
+					let parsedJSON = null;
+					try {
+						parsedJSON = JSON.parse(str);
+					}
+					catch (e) {
+						// ignore the error for now
+					}
+
+					resolve(parsedJSON);
 				});
 
 				res.on('error', function (err) {