Skip to content

Commit

Permalink
[terraform] downgrade non-server errors to debug
Browse files Browse the repository at this point in the history
Summary: These are quite noisy and spam our tunnelbroker error logs. This diff downgrades these to debug

Test Plan: cargo check

Reviewers: kamil, bartek

Reviewed By: bartek

Subscribers: ashoat, tomek

Differential Revision: https://phab.comm.dev/D13463
  • Loading branch information
wyilio committed Sep 26, 2024
1 parent b947b11 commit 1d09ec4
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions services/tunnelbroker/src/websockets/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,7 @@ async fn send_error_init_response(
if let Err(send_error) =
outgoing.send(Message::Text(serialized_response)).await
{
error!("Failed to send init error response: {:?}", send_error);
debug!("Failed to send init error response: {:?}", send_error);
}
}
Err(ser_error) => {
Expand Down Expand Up @@ -212,13 +212,13 @@ async fn accept_connection(
session
}
Err((err, outgoing)) => {
error!("Failed to create session with device");
debug!("Failed to create session with device");
send_error_init_response(err, outgoing).await;
return;
}
}
} else {
error!("Failed to create session with device");
debug!("Failed to create session with device");
send_error_init_response(SessionError::InvalidMessage, outgoing).await;
return;
};
Expand Down
2 changes: 1 addition & 1 deletion services/tunnelbroker/src/websockets/session.rs
Original file line number Diff line number Diff line change
Expand Up @@ -210,7 +210,7 @@ pub async fn initialize_amqp(
handle_first_message_from_device(&payload).await?
}
_ => {
error!("Client sent wrong frame type for establishing connection");
debug!("Client sent wrong frame type for establishing connection");
return Err(SessionError::InvalidMessage);
}
};
Expand Down

0 comments on commit 1d09ec4

Please sign in to comment.