Skip to content

Commit

Permalink
Remove backward incompatible change
Browse files Browse the repository at this point in the history
Signed-off-by: Nikita Usatov <[email protected]>
  • Loading branch information
nikitakosatka committed Dec 23, 2024
1 parent 5d1ecea commit a2d2050
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion dhcpv4/dhcpv4.go
Original file line number Diff line number Diff line change
Expand Up @@ -606,7 +606,7 @@ func (d *DHCPv4) NTPServers() []net.IP {
//
// The NetBIOS over TCP/IP Name Server option is described by RFC 2132, Section 8.5.
func (d *DHCPv4) NetBIOSNameServers() []net.IP {
return GetIPs(OptionNetBIOSOverTCPIPNameServers, d.Options)
return GetIPs(OptionNetBIOSOverTCPIPNameServer, d.Options)
}

// DNS parses the DHCPv4 Domain Name Server option if present.
Expand Down
2 changes: 1 addition & 1 deletion dhcpv4/option_ips.go
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ func OptNTPServers(ntpServers ...net.IP) Option {
// The NetBIOS over TCP/IP Name Server option is described by RFC 2132, Section 8.5.
func OptNetBIOSNameServers(netBIOSNameServers ...net.IP) Option {
return Option{
Code: OptionNetBIOSOverTCPIPNameServers,
Code: OptionNetBIOSOverTCPIPNameServer,
Value: IPs(netBIOSNameServers),
}
}
Expand Down
4 changes: 2 additions & 2 deletions dhcpv4/option_ips_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -71,9 +71,9 @@ func TestGetNTPServers(t *testing.T) {

func TestOptNetBIOSNameServers(t *testing.T) {
o := OptNetBIOSNameServers(net.IPv4(192, 168, 0, 1), net.IPv4(192, 168, 0, 10))
require.Equal(t, OptionNetBIOSOverTCPIPNameServers, o.Code)
require.Equal(t, OptionNetBIOSOverTCPIPNameServer, o.Code)
require.Equal(t, []byte{192, 168, 0, 1, 192, 168, 0, 10}, o.Value.ToBytes())
require.Equal(t, "NetBIOS over TCP/IP Name Servers: 192.168.0.1, 192.168.0.10", o.String())
require.Equal(t, "NetBIOS over TCP/IP Name Server: 192.168.0.1, 192.168.0.10", o.String())
}

func TestGetNetBIOSNameServers(t *testing.T) {
Expand Down
4 changes: 2 additions & 2 deletions dhcpv4/types.go
Original file line number Diff line number Diff line change
Expand Up @@ -180,7 +180,7 @@ const (
OptionNetworkInformationServers optionCode = 41
OptionNTPServers optionCode = 42
OptionVendorSpecificInformation optionCode = 43
OptionNetBIOSOverTCPIPNameServers optionCode = 44
OptionNetBIOSOverTCPIPNameServer optionCode = 44
OptionNetBIOSOverTCPIPDatagramDistributionServer optionCode = 45
OptionNetBIOSOverTCPIPNodeType optionCode = 46
OptionNetBIOSOverTCPIPScope optionCode = 47
Expand Down Expand Up @@ -345,7 +345,7 @@ var optionCodeToString = map[OptionCode]string{
OptionNetworkInformationServers: "Network Information Servers",
OptionNTPServers: "NTP Servers",
OptionVendorSpecificInformation: "Vendor Specific Information",
OptionNetBIOSOverTCPIPNameServers: "NetBIOS over TCP/IP Name Servers",
OptionNetBIOSOverTCPIPNameServer: "NetBIOS over TCP/IP Name Server",
OptionNetBIOSOverTCPIPDatagramDistributionServer: "NetBIOS over TCP/IP Datagram Distribution Server",
OptionNetBIOSOverTCPIPNodeType: "NetBIOS over TCP/IP Node Type",
OptionNetBIOSOverTCPIPScope: "NetBIOS over TCP/IP Scope",
Expand Down

0 comments on commit a2d2050

Please sign in to comment.