Merge pull request #2420 from atom-community/ansiFilter.toHtml

fix: try catch ansi-to-html

lib/script-view.js

@@ -173,7 +173,13 @@ export default class ScriptView extends MessagePanelView {
       line = _.escape(line);
     }
 
-    line = this.ansiFilter.toHtml(line);
+    try {
+      line = this.ansiFilter.toHtml(line);
+    } catch (e) {
+      // TODO why this happens https://github.com/atom-community/atom-script/issues/2358
+      console.error(e);
+    }
+
     line = linkPaths(line);
 
     const { clientHeight, scrollTop, scrollHeight } = this.body[0];

spec/fixtures/issue_2358.py

@@ -0,0 +1,9 @@
+# See the console after running this
+
+import requests
+
+head = {'User-Agent': 'Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Ubuntu Chromium/65.0.3325.181 Chrome/65.0.3325.181 Safari/537.36'}
+
+r = requests.get('https://www.oxfordlearnersdictionaries.com/us/wordlists/oxford3000-5000', headers=head)
+
+print(r.text)