Skip to content

Commit 1a6cbf5

Browse files
authored
Merge pull request #1579 from gravitl/bugfix_v0.16.0_gw_compare
fixing compare
2 parents c86e32d + e93eaf8 commit 1a6cbf5

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

netclient/functions/mqhandlers.go

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -216,10 +216,13 @@ func UpdatePeers(client mqtt.Client, msg mqtt.Message) {
216216
}
217217
//check if internet gateway has changed
218218
oldGateway, err := net.ResolveUDPAddr("udp", cfg.Node.InternetGateway)
219+
220+
// note: may want to remove second part (oldGateway == &net.UDPAddr{})
221+
// since it's a pointer, will never be true
219222
if err != nil || (oldGateway == &net.UDPAddr{}) {
220223
oldGateway = nil
221224
}
222-
if (internetGateway == nil && oldGateway != nil) || (internetGateway != nil && internetGateway != oldGateway) {
225+
if (internetGateway == nil && oldGateway != nil) || (internetGateway != nil && internetGateway.String() != oldGateway.String()) {
223226
cfg.Node.InternetGateway = internetGateway.String()
224227
if err := config.ModNodeConfig(&cfg.Node); err != nil {
225228
logger.Log(0, "failed to save internet gateway", err.Error())

0 commit comments

Comments
 (0)