Skip to content

Commit 90a1b56

Browse files
authored
fix: move metrics to all messages (dfinity#3690)
Before this, only `getData` and `getHeaders` messages were recorded in the metrics. With this change, all messages should be recorded.
1 parent c8a571c commit 90a1b56

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

rs/bitcoin/adapter/src/connectionmanager.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -394,6 +394,10 @@ impl ConnectionManager {
394394
addr: &SocketAddr,
395395
network_message: NetworkMessage,
396396
) -> ConnectionManagerResult<()> {
397+
self.metrics
398+
.bitcoin_messages_sent
399+
.with_label_values(&[network_message.cmd()])
400+
.inc();
397401
let conn = self.get_connection(addr)?;
398402
if conn.send(network_message).is_err() {
399403
conn.disconnect();
@@ -586,10 +590,6 @@ impl ConnectionManager {
586590

587591
impl Channel for ConnectionManager {
588592
fn send(&mut self, command: Command) -> Result<(), ChannelError> {
589-
self.metrics
590-
.bitcoin_messages_sent
591-
.with_label_values(&[command.message.cmd()])
592-
.inc();
593593
let Command { address, message } = command;
594594
if let Some(addr) = address {
595595
self.send_to(&addr, message).ok();

0 commit comments

Comments
 (0)