Skip to content

Commit

Permalink
protocols/relay: Do not close connection on upgrade timeouts
Browse files Browse the repository at this point in the history
  • Loading branch information
mxinden committed Mar 10, 2021
1 parent 7956f0b commit d55bd99
Showing 1 changed file with 3 additions and 11 deletions.
14 changes: 3 additions & 11 deletions protocols/relay/src/handler.rs
Original file line number Diff line number Diff line change
Expand Up @@ -391,10 +391,7 @@ impl ProtocolsHandler for RelayHandler {
error: ProtocolsHandlerUpgrErr<protocol::RelayListenError>,
) {
match error {
ProtocolsHandlerUpgrErr::Timeout => {
self.pending_error = Some(ProtocolsHandlerUpgrErr::Timeout);
}
ProtocolsHandlerUpgrErr::Timer => {}
ProtocolsHandlerUpgrErr::Timeout | ProtocolsHandlerUpgrErr::Timer => {}
ProtocolsHandlerUpgrErr::Upgrade(upgrade::UpgradeError::Select(
upgrade::NegotiationError::Failed,
)) => {}
Expand Down Expand Up @@ -426,10 +423,7 @@ impl ProtocolsHandler for RelayHandler {
request_id,
} => {
match error {
ProtocolsHandlerUpgrErr::Timeout => {
self.pending_error = Some(ProtocolsHandlerUpgrErr::Timeout);
}
ProtocolsHandlerUpgrErr::Timer => {}
ProtocolsHandlerUpgrErr::Timeout | ProtocolsHandlerUpgrErr::Timer => {}
ProtocolsHandlerUpgrErr::Upgrade(upgrade::UpgradeError::Select(
upgrade::NegotiationError::Failed,
)) => {}
Expand Down Expand Up @@ -509,11 +503,9 @@ impl ProtocolsHandler for RelayHandler {
..
} => {
let err_code = match error {
ProtocolsHandlerUpgrErr::Timeout => {
self.pending_error = Some(ProtocolsHandlerUpgrErr::Timeout);
ProtocolsHandlerUpgrErr::Timeout | ProtocolsHandlerUpgrErr::Timer => {
circuit_relay::Status::HopCantOpenDstStream
}
ProtocolsHandlerUpgrErr::Timer => circuit_relay::Status::HopCantOpenDstStream,
ProtocolsHandlerUpgrErr::Upgrade(upgrade::UpgradeError::Select(
upgrade::NegotiationError::Failed,
)) => circuit_relay::Status::HopCantSpeakRelay,
Expand Down

0 comments on commit d55bd99

Please sign in to comment.