search

CyberShadow / dhcptest

Cross-platform DHCP test client
https://blog.cy.md/2013/01/10/dhcp-test-client
366 stars 57 forks source link

Option 121 (Classless static route) doesn't exist #29

Closed Dronec closed 1 year ago

Dronec commented 1 year ago

I've already made an implementation in my local branch, happy to create a PR

CyberShadow commented 1 year ago

OK, please do!

sjackson0109 commented 1 year ago

If it helps..I’d like to see option 191, 192, 176, 142 and 120. #telephonyPerhaps 204, 129-135, 67, 66, 60 and 42 #pxeUsed this software hundereds of times. It’s increadibly useful for network engineers supporting pxe boot and telephony service/handset deployments. Highly recommend it!SimonOn 19 Apr 2023, at 14:27, Vladimir Panteleev @.***> wrote: OK, please do!

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

CyberShadow commented 1 year ago

If it's in a RFC, let's add it.

For unofficial extensions, we should be more careful.

67, 66, 60 and 42 are already implemented.

Dronec commented 1 year ago

OK, please do!

I'll need contributor access to your repo image

CyberShadow commented 1 year ago

On GitHub, to contribute to a public project, you fork it first, then push the changes to a branch on your fork. Then you can create a PR there.

https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request-from-a-fork