fix: strip Bom before parse to json

axios · Jul 2, 2020 · c522127 · c522127
1 parent deb914d
commit c522127
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/lib/adapters/http.js b/lib/adapters/http.js
@@ -235,14 +235,13 @@ module.exports = function httpAdapter(config) {
           var responseData = Buffer.concat(responseBuffer);
           if (config.responseType !== 'arraybuffer') {
             responseData = responseData.toString(config.responseEncoding);
+            if (!config.responseEncoding || config.responseEncoding === 'utf8') {
+              responseData = utils.stripBOM(responseData);
+            }
             // only response type 'json' return JSON response,  'text' and '' return string response
             if (config.responseType === 'json') {
               responseData = utils.tryParseJSON(responseData);
             }
-
-            if (!config.responseEncoding || config.responseEncoding === 'utf8') {
-              responseData = utils.stripBOM(responseData);
-            }
           }
 
           response.data = responseData;