Skip to content

Commit

Permalink
Update the comments and logs
Browse files Browse the repository at this point in the history
  • Loading branch information
djordon committed Feb 4, 2025
1 parent f4f62e6 commit d5730c4
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 8 deletions.
4 changes: 2 additions & 2 deletions signer/src/block_observer.rs
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,7 @@ where
tracing::warn!(%error, "could not process stacks blocks");
}

tracing::debug!("Updating the signer state");
tracing::debug!("updating the signer state");
if let Err(error) = self.update_signer_state(block_hash).await {
tracing::warn!(%error, "could not update the signer state");
continue;
Expand Down Expand Up @@ -625,7 +625,7 @@ impl<C: Context, B> BlockObserver<C, B> {
tracing::info!("loading sbtc limits from Emily");
self.update_sbtc_limits().await?;

tracing::info!("Updating the signer state with the current signer set");
tracing::info!("updating the signer state with the current signer set");
self.set_signer_set_and_aggregate_key(chain_tip).await
}
}
Expand Down
11 changes: 5 additions & 6 deletions signer/tests/integration/block_observer.rs
Original file line number Diff line number Diff line change
Expand Up @@ -852,12 +852,11 @@ async fn next_headers_to_process_ignores_known_headers() {
testing::storage::drop_db(db).await;
}

/// The [`TxCoordinatorEventLoop::get_signer_set_and_aggregate_key`]
/// function is supposed to fetch the "current" signing set and the
/// aggregate key to use for bitcoin transactions. It attempts to get the
/// latest rotate-keys contract call transaction confirmed on the canonical
/// Stacks blockchain and falls back to the DKG shares table if no such
/// transaction can be found.
/// The [`get_signer_set_and_aggregate_key`] function is supposed to fetch
/// the "current" signing set and the aggregate key to use for bitcoin
/// transactions. It attempts to get the latest rotate-keys contract call
/// transaction confirmed on the canonical Stacks blockchain and falls back
/// to the DKG shares table if no such transaction can be found.
///
/// This tests that we prefer rotate keys transactions if it's available
/// but will use the DKG shares behavior is indeed the case.
Expand Down

0 comments on commit d5730c4

Please sign in to comment.