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

Transition from the lwip crate to a native alternative #64

Closed
SajjadPourali opened this issue Oct 17, 2023 · 6 comments
Closed

Transition from the lwip crate to a native alternative #64

SajjadPourali opened this issue Oct 17, 2023 · 6 comments
Assignees
Labels
C-feature Category: feature. This is adding a new feature. E-help-wanted Effort: Call for participation: Help is requested to fix this issue.
Milestone

Comments

@SajjadPourali
Copy link
Contributor

LwIP is used as a TCP stack for TUN functionality. I've noticed that this library works; however, it is not suitable for Narrowlink in the long term due to three main problems:

  1. it's difficult to handle large packets (which causes issues when integrating with WinTun)
  2. troubleshooting difficulties
  3. overhead
@SajjadPourali SajjadPourali added C-feature Category: feature. This is adding a new feature. E-help-wanted Effort: Call for participation: Help is requested to fix this issue. labels Oct 17, 2023
@SajjadPourali SajjadPourali added this to the 0.4.0 milestone Oct 17, 2023
@SajjadPourali SajjadPourali self-assigned this Oct 17, 2023
@SajjadPourali SajjadPourali changed the title Transition from the lwip crate to an native alternative Transition from the lwip crate to a native alternative Oct 17, 2023
@SajjadPourali
Copy link
Contributor Author

smoltcp might be a good option if the following pull request get merged.

smoltcp-rs/smoltcp#814
smoltcp-rs/smoltcp#809

@9to1url
Copy link

9to1url commented Oct 21, 2023

Great, I am also posted question related to lwIP in another thread.

@SajjadPourali
Copy link
Contributor Author

https://crates.io/crates/ipstack

@9to1url
Copy link

9to1url commented Oct 22, 2023

Cool 😎

@SajjadPourali
Copy link
Contributor Author

SajjadPourali commented Oct 23, 2023

#73 - It requires testing ... .

@SajjadPourali SajjadPourali modified the milestones: 0.4.0, 0.2.0 Oct 23, 2023
@SajjadPourali
Copy link
Contributor Author

#73

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-feature Category: feature. This is adding a new feature. E-help-wanted Effort: Call for participation: Help is requested to fix this issue.
Projects
None yet
Development

No branches or pull requests

2 participants