process 'error' event is undocumented

[addaleax]

node/lib/internal/event_target.js

Line 604 in 8b8620d

process.emit('error', err, event);

/cc @benjamingr

[benjamingr]

I wonder if we want to keep it or emit uncaughtException directly since no-one actually complained about this or asked for this since EventEmitter was added.

We can also just doc it.

[benjamingr]

Any strong opinions?

[addaleax]

I was definitely caught by surprise by this being added, and would be happy to treat it as uncaught exceptions (i.e. act as if there’s no 'error' listener present).

[jasnell]

What's the issue here? There is documentation for this: https://nodejs.org/dist/latest-v15.x/docs/api/events.html#events_eventtarget_error_handling

[addaleax]

https://nodejs.org/api/process.html#process_process_events is missing them – we always document events on the emitter.

[benjamingr]

@jasnell any strong opinions about making this an uncaughtException? I'm happy to make a PR.

[jasnell]

About making what an uncaughtException? Sorry, it's not clear

[addaleax]

@jasnell About making an exception inside a EventTarget event handler be treated like an uncaught exception, i.e. trigger the uncaughtException listener if there is one and otherwise exit the process with error code 1.

[Linkgoron]

I'll try taking a stab at this