Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(swarm): better error message for failed DialPeerConditions #4409

Merged
merged 10 commits into from
Sep 6, 2023
Merged

fix(swarm): better error message for failed DialPeerConditions #4409

Changes from 2 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
951889a
fix(swarm): improve error message on false DialPeerCondition
chirag-bgh Aug 29, 2023
5c9e96b
expand PeerCondition arm
chirag-bgh Aug 29, 2023
7b369ea
Update swarm/src/lib.rs
chirag-bgh Sep 4, 2023
33a4cca
Update swarm/src/lib.rs
chirag-bgh Sep 4, 2023
597ddb6
Merge branch 'master' into swarm_error_msg
chirag-bgh Sep 4, 2023
8922d10
fix
chirag-bgh Sep 4, 2023
9c475b9
bumb libp2p-swarm
chirag-bgh Sep 5, 2023
fb907e0
nit
chirag-bgh Sep 5, 2023
6a48e8d
Update swarm/CHANGELOG.md
thomaseizinger Sep 5, 2023
c97a2b6
Merge branch 'master' into swarm_error_msg
thomaseizinger Sep 6, 2023
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
6 changes: 4 additions & 2 deletions swarm/src/lib.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1580,8 +1580,10 @@ impl fmt::Display for DialError {
f,
"Dial error: tried to dial local peer id at {endpoint:?}."
),
DialError::DialPeerConditionFalse(c) => {
write!(f, "Dial error: condition {c:?} for dialing peer was false.")
DialError::DialPeerConditionFalse(condition) => match condition {
chirag-bgh marked this conversation as resolved.
Show resolved Hide resolved
PeerCondition::Disconnected => write!(f, "Dial error: condition dial was configured to only happen when disconnected (`PeerCondition::Disconnected`), but node is already connected, thus cancelling new dial."),
PeerCondition::NotDialing => write!(f, "Dial error: condition dial was configured to only if there is currently no ongoing dialing attempt (`PeerCondition::NotDialing`), but node is already dialing, thus cancelling new dial."),
chirag-bgh marked this conversation as resolved.
Show resolved Hide resolved
PeerCondition::Always => write!(f, "Dial error: condition dial was configured to always happen (`PeerCondition::Always`)."),
chirag-bgh marked this conversation as resolved.
Show resolved Hide resolved
}
DialError::Aborted => write!(
f,
Expand Down