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

Switch to net/ipv4 package #1

Open
ppacher opened this issue Sep 15, 2019 · 0 comments
Open

Switch to net/ipv4 package #1

ppacher opened this issue Sep 15, 2019 · 0 comments
Assignees
Labels
help wanted Extra attention is needed improvement New feature or request
Projects

Comments

@ppacher
Copy link
Collaborator

ppacher commented Sep 15, 2019

It should be possible to use the net/ipv4 packet for the raw socket handling (currently we are using a third-party lib). Check how much efford it would be and how we could migrate to the std lib. This will likely also add support for Windows and BSD variants.
See https://godoc.org/golang.org/x/net/ipv4#RawConn
However, it seems like we will then not be able to send unicasts to specific L2 addresses as required by RFC (for example, DHCPACK is directly sent to yiaddr and the clients hwaddr without and ARP requests). However, we may still use the net/ipv4 package to support Windows and BSD variants with the limitation that those unicasts must be broadcasted. According to RFC this is allowed as well if unicasting isn't possible

@ppacher ppacher added the improvement New feature or request label Sep 15, 2019
@ppacher ppacher self-assigned this Sep 15, 2019
@ppacher ppacher added this to In progress in NextDHCP Sep 15, 2019
@ppacher ppacher moved this from In progress to Reviewer approved in NextDHCP Sep 15, 2019
@ppacher ppacher moved this from Reviewer approved to In progress in NextDHCP Sep 15, 2019
@ppacher ppacher moved this from In progress to Todo in NextDHCP Sep 15, 2019
@ppacher ppacher added the help wanted Extra attention is needed label Sep 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed improvement New feature or request
Projects
NextDHCP
  
Todo
Development

No branches or pull requests

1 participant