Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

death listener registration error #5

Closed
emarteca opened this issue Jan 17, 2020 · 2 comments
Closed

death listener registration error #5

emarteca opened this issue Jan 17, 2020 · 2 comments

Comments

@emarteca
Copy link

Hi all, I've been working on a tool to identify instances of events registered to the wrong object in uses of some JavaScript event-driven APIs, as part of a research project.
The tool flagged line 49 in example/server.js, on the registration of the listener for death.

The reason I believe this is an error is as follows (from looking at the nodejs cluster API documentation):
the listener for death is registered on the return from the call to cluster.fork().
However, death is not an event on cluster.Worker.

Thanks!

@indutny
Copy link
Owner

indutny commented Jan 17, 2020

Hey!

This is a very very old module and it uses outdated event names. I don't think that anyone uses it nor that it compiles for the recent Node.js versions.

Thank you for reaching out and good luck with your research!

@indutny indutny closed this as completed Jan 17, 2020
@emarteca
Copy link
Author

Hey, thanks for the speedy response, and for letting me know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants