Implementing getSubOptionValue about net-dhcp HOT 6 OPEN

The code there is just copied from getOptionValue, i havent gotten around to reworking it to decode suboptions.

from net-dhcp.

Alright. I can do that and submit a PR.

from net-dhcp.

I would be delighted to accept the PR

from net-dhcp.

Just FYI There are packet captures of option 82 in the t/data directory and tests that are already examining those captures. Both would be a great lace to tune your code. If you have more exotic captures of option 82 thy would be great to have added!

from net-dhcp.

It seems like RFC 3046 does not define what data types individual suboptions are required to take. Our use-case for it requires that suboption 1 (client id) is a string, but the pcap data files in the t/ directory have suboption 1 values that are clearly hex (leading null bytes).

Luckily, people can just use getSubOptionRaw() to get the raw string version, and getSubOptionValue() to get the hex value, but what about any other format? Do we just make people do the thing themselves?

from net-dhcp.

Youre seeing why its not implemented :)

Have a go at it as best you can, and in general I just see what wireshark does with the packet then take that as being correct.

from net-dhcp.