Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

client/authority-discovery: Append PeerId to Multiaddr at most once #6933

Merged
bkchr merged 4 commits into from
Aug 24, 2020
Merged

client/authority-discovery: Append PeerId to Multiaddr at most once #6933

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
95e948f
client/authority-discovery/worker: Extract address getter
mxinden Aug 21, 2020
cdd0ecf
client/authority-discovery: Test for no duplicate p2p components
mxinden Aug 21, 2020
5d63b1d
client/authority-discovery: Append PeerId to Multiaddr at most once
mxinden Aug 21, 2020
7d51f6c
client/authority-discovery: Remove explicit return
mxinden Aug 24, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
client/authority-discovery: Remove explicit return
  • Loading branch information
@mxinden
mxinden committed Aug 24, 2020
commit 7d51f6cb6727eb8500683c912645a09788af7baf
5 changes: 3 additions & 2 deletions client/authority-discovery/src/worker.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -243,9 +243,10 @@ where
.into_iter()
.map(move |a| {
if a.iter().any(|p| matches!(p, multiaddr::Protocol::P2p(_))) {
return a;
a
} else {
a.with(multiaddr::Protocol::P2p(peer_id.clone()))
}
a.with(multiaddr::Protocol::P2p(peer_id.clone()))
}),
)
}
Expand Down