server: use deduped addrMap for connReq creation

Use the addrMap for connReq creation to prevent creating duplicate
connection requests if persistentPeerAddrs contains duplicate addresses.
This commit is contained in:
Elle Mouton 2021-10-07 13:32:02 +02:00
parent dd3719d5de
commit 84f6b70773
No known key found for this signature in database
GPG Key ID: D7D916376026F177
2 changed files with 4 additions and 5 deletions

View File

@ -460,6 +460,9 @@ you.
certain channels couldn't be passed to `lncli getchaninfo` due to their 8-byte
compact ID being too large for an int64.
* [Dedup stored peer addresses before creating connection requests to prevent
redundant connection requests](https://github.com/lightningnetwork/lnd/pull/5839)
## Documentation
The [code contribution guidelines have been updated to mention the new

View File

@ -3772,11 +3772,7 @@ func (s *server) connectToPersistentPeer(pubKeyStr string) {
// Any addresses left in addrMap are new ones that we have not made
// connection requests for. So create new connection requests for those.
for _, addr := range s.persistentPeerAddrs[pubKeyStr] {
if _, ok := addrMap[addr.String()]; !ok {
continue
}
for _, addr := range addrMap {
connReq := &connmgr.ConnReq{
Addr: addr,
Permanent: true,