This repository has been archived by the owner on May 18, 2024. It is now read-only.

Syntax errors should be thrown before calling reviver in polyfill #4

Open

zloirock opened this issue Jan 28, 2023 · 0 comments

zloirock commented Jan 28, 2023 •

edited

Loading

It's not related to the JSON standard, but for polyfill by the ES spec JSON should be parsed (validated) before calling reviver.

JSON.parse("[1]{", (a, b, c) => console.log("should not be called")); // => "should not be called"

https://tc39.es/ecma262/#sec-json.parse

zloirock mentioned this issue

In case of polyfill, reviver function should be called in the holder context #5

Open

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.