logos-storage/logos-storage-nim
1
0
Fork 0
mirror of https://github.com/logos-storage/logos-storage-nim.git synced 2026-06-28 05:19:28 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix clientMode order and avoid retry port mapping after Not Reachable

Browse Source
This commit is contained in:
Arnaud 2026-06-01 17:44:11 +04:00
parent 1193a2c29d
commit 00e6557052
No known key found for this signature in database
GPG Key ID: A6C7C781817146FA
1 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
storage/nat.nim
View File

@ -154,8 +154,10 @@ method handleNatStatus*(
else:
debug "AutoRelayService stopped"
discovery.protocol.clientMode = false
# Update the record first, then flip to server mode: otherwise the node
# briefly serves DHT queries with the previous (possibly empty) record.
discovery.updateRecordsAndSpr(@[dialBackAddr.get], udpPort = discoveryPort)
discovery.protocol.clientMode = false
of NotReachable:
var hasPortMapping = false
@ -173,12 +175,18 @@ method handleNatStatus*(
# The mapping was created the the node is still not reachable.
# In that case, we delete the mapping and relay will start.
# We will keep retrying on the next iteration
m.close()
# We remove the announced records.
# Eventually, it will we updated by the relay when it started
discovery.updateRecordsAndSpr(@[], udpPort = discoveryPort)
elif autoRelayService.isRunning:
# The mapping was already tried and did not make the node reachable.
# If the relay is running, there is nothing to do.
# We do not want to retry the port mapping if it failed already,
# it would stop the relay service while there is little chance to have
# a Reachable status after it was detected Not Reachable the first time.
discard
else:
debug "Node is not reachable trying port mapping now"