PerimeterX/restringer

Display Process Uncaught Exceptions

BenBaryoPX opened this issue · 0 comments

@lanvent writes (#85)

The following code snippet in the library:

  // Silence asyc errors
  process.on('uncaughtException', () => {});

While I understand that the intent is to silence asynchronous errors, this also suppresses all exceptions. It made debugging a bit challenging for me as I am not deeply familiar with JavaScript. It might be helpful to add an option or warning for this behavior and some guidance on error logging.

I suggest keeping this behavior as default, and adding an option to disable the error suppression (perhaps using process.removeListener).