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

Throws on node 18 #39

Closed
Alexander-Taran opened this issue Jun 21, 2022 · 0 comments · Fixed by #40
Closed

Throws on node 18 #39

Alexander-Taran opened this issue Jun 21, 2022 · 0 comments · Fixed by #40

Comments

@Alexander-Taran
Copy link
Contributor

Alexander-Taran commented Jun 21, 2022

There is a breaking change in node 18
family property is now number not a string

nodejs/node#42787

I'd throw up a PR but I'm not sure how to treat number.
Weather it is ok just to .toString() it or "IPv" + address.family

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

Successfully merging a pull request may close this issue.

2 participants