patriksimek/vm2

Add support for "correct" stack traces for errors in the sandbox

Open

#87 aperta il 4 ago 2017

Vedi su GitHub
 (13 commenti) (8 reazioni) (0 assegnatari)JavaScript (285 fork)batch import
feature requesthelp wanted

Metriche repository

Star
 (3798 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

It would be nice if vm2 could display "correct" stack traces when an error is thrown from the sandbox.

For instance, this is what is currently displayed:

/home/user/box-js/node_modules/vm2/lib/main.js:213
                        throw this._internal.Decontextify.value(e);
                        ^

Error: foobar
    at Object.log (/home/user/box-js/analyze.js:248:10)
    at Object.apply (/home/user/box-js/node_modules/vm2/lib/contextify.js:288:34)
    at vm.js:491:9
    at ContextifyScript.Script.runInContext (vm.js:53:29)
    at VM.run (/home/user/box-js/node_modules/vm2/lib/main.js:207:72)
    at Object.<anonymous> (/home/user/box-js/analyze.js:383:5)
    at Module._compile (module.js:569:30)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:503:32)
    at tryModuleLoad (module.js:466:12)

Is there a way to figure out which line (in the sandboxed code) was responsible for calling the function that threw the error?

Guida contributor

Direzione di ricerca
Investiga come vm2 crea gli stack trace per gli errori lanciati nella sandbox e modifica il metodo Decontextify.value() o Error.prepareStackTrace per preservare i numeri di riga originali della sandbox. Guarda i file contextify.js e main.js per comprendere il processo di wrapping degli errori.
Tech stack
javascriptnodejs
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Node.jsJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65