Skip to content

src: fix JSONParser leaking internal V8 scopes

Rodrigo Muino Tomonari requested to merge github/fork/kvakil/refactor-json-parser into main

JSONParser uses V8's JSON.parse (for now), meaning that its uses handles and contexts. JSONParser was leaking its internal HandleScope and Context::Scope.

Move the scope construction to the member functions to prevent those scopes from leaking.

Refs: https://github.com/nodejs/node/pull/50680#discussion_r1390363367

Merge request reports