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

next version #288

Open
mildsunrise opened this issue Sep 24, 2021 · 1 comment
Open

next version #288

mildsunrise opened this issue Sep 24, 2021 · 1 comment

Comments

@mildsunrise
Copy link
Contributor

mildsunrise commented Sep 24, 2021

(this is mostly a reminder for myself, you can ignore it if you want)

I'm considering making a rewrite + major version of this module, with the usual good practices for native addons.
Goals are:

  • Bundle libpcap, link statically.
  • Prebuilds for armv7, arm64, x86, x64 and for glibc and musl.
  • Use N-API, be context safe.
  • Avoid using uv_poll_t directly.
  • Rewrite in TypeScript.
  • Prevent loop starvation.
  • Move dissectors to other module; people may want to use the dissectors without the native addon and vice-versa.
  • Change API, make it mirror dgram.Socket and otherwise libpcap: use camelCase, enums, expose setters (allow changing settings without reopening session), expose ref/unref, explicit start / stop / close

Edit: A user reached out to me about this, so I wanted to point out that I'm not currently working on this and happy about other people taking over. If you do start work on these points, please comment or reference this issue since I might still do them if I find time.

@herenickname
Copy link

@mildsunrise hi! are there any updates?

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