fix: Do not use the native parser in Safari and Node.js 4

It does not return the offset of the error occurrence, which would not allow adding the context information to the thrown error.
prantlf · Jun 2, 2019 · a4a606c · a4a606c
1 parent 03561ec
commit a4a606c
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/configurable-parser.js b/src/configurable-parser.js
@@ -39,9 +39,12 @@ function restoreContext (changed) {
   }
 }
 
+var isSafari = typeof navigator !== 'undefined' && /Safari/.test(navigator.userAgent) && /Apple Computer/.test(navigator.vendor)
+var oldNode = typeof process !== 'undefined' && process.version.startsWith('v4.')
+
 function needsCustomParser () {
   return this.ignoreComments || this.allowSingleQuotedStrings ||
-    this.mode === 'cjson' || this.mode === 'json5'
+    this.mode === 'cjson' || this.mode === 'json5' || isSafari || oldNode
 }
 
 function getReviver (options) {