fix #14

Huachao · Jul 19, 2016 · 3e8b2af · 3e8b2af
1 parent 03bc69b
commit 3e8b2af
Showing 1 changed file with 15 additions and 2 deletions.
diff --git a/src/views/httpResponseTextDocumentContentProvider.ts b/src/views/httpResponseTextDocumentContentProvider.ts
@@ -1,6 +1,6 @@
 "use strict";
 
-import { TextDocumentContentProvider, EventEmitter, Event, Uri } from 'vscode';
+import { TextDocumentContentProvider, EventEmitter, Event, Uri, window } from 'vscode';
 import { HttpResponse } from '../models/httpResponse'
 import { MimeUtility } from '../mimeUtility'
 
@@ -66,7 +66,11 @@ ${HttpResponseTextDocumentContentProvider.formatBody(this.response.body, this.re
         if (contentType) {
             let type = MimeUtility.parse(contentType).type;
             if (type === 'application/json') {
-                body = JSON.stringify(JSON.parse(body), null, 4);
+                if (HttpResponseTextDocumentContentProvider.isJsonString(body)) {
+                    body = JSON.stringify(JSON.parse(body), null, 4);
+                 } else {
+                    window.showWarningMessage('The content type of response is application/json, while response body is not a valid json string');
+                 }
             } else if (type === 'application/xml') {
                 body = pd.xml(body);
             }
@@ -80,4 +84,13 @@ ${HttpResponseTextDocumentContentProvider.formatBody(this.response.body, this.re
             return HttpResponseTextDocumentContentProvider._tagsToReplace[tag] || tag;
         });
     }
+
+    private static isJsonString(data: string) {
+        try {
+            JSON.parse(data);
+            return true;
+        } catch (e) {
+            return false;
+        }
+    }
 }