Update server_nat.go

This commit is contained in:
Felix Lange 2025-04-09 00:08:13 +02:00 committed by GitHub
parent d984c79585
commit a6e7844c8d
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -164,7 +164,7 @@ func (srv *Server) portMappingLoop() {
// larger than the retry interval, this does not mean we lost our existing // larger than the retry interval, this does not mean we lost our existing
// mapping. We do not reset the external port, as it is still our best chance, // mapping. We do not reset the external port, as it is still our best chance,
// but we do retry soon. // but we do retry soon.
// TODO: we could check the error code, but again, UPnP implementations are buggy. // We could check the error code, but again, UPnP implementations are buggy.
log.Debug("Couldn't refresh port mapping", "err", err) log.Debug("Couldn't refresh port mapping", "err", err)
m.retries++ m.retries++
if m.retries > maxRetries { if m.retries > maxRetries {
@ -175,7 +175,8 @@ func (srv *Server) portMappingLoop() {
} }
} }
m.nextTime = srv.clock.Now().Add(portMapRetryInterval) m.nextTime = srv.clock.Now().Add(portMapRetryInterval)
continue //TODO: this means we never reset the ENR. Is that what we want? // Note ENR is not updated here, i.e. we keep the last port.
continue
} }
// It was mapped! // It was mapped!