fix: unref nat manager retries (#877)

The retry operation in the NAT Manager can prevent node from shutting
down, so unref the retries so they don't keep adding work to the
event loop.
This commit is contained in:
Alex Potsides
2021-01-29 13:09:59 +00:00
committed by GitHub
parent a64c02838c
commit ce2a624a09

View File

@ -138,9 +138,9 @@ class NatManager {
this._client = { this._client = {
// these are all network operations so add a retry // these are all network operations so add a retry
map: (...args) => retry(() => map(...args), { onFailedAttempt: log.error }), map: (...args) => retry(() => map(...args), { onFailedAttempt: log.error, unref: true }),
destroy: (...args) => retry(() => destroy(...args), { onFailedAttempt: log.error }), destroy: (...args) => retry(() => destroy(...args), { onFailedAttempt: log.error, unref: true }),
externalIp: (...args) => retry(() => externalIp(...args), { onFailedAttempt: log.error }) externalIp: (...args) => retry(() => externalIp(...args), { onFailedAttempt: log.error, unref: true })
} }
return this._client return this._client