pezkuwichain/pezkuwi-subxt
1
0
Fork 0
mirror of https://github.com/pezkuwichain/pezkuwi-subxt.git synced 2026-06-24 10:11:08 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add Logging to Finality Verifier Pallet (#702)

Browse Source 
* Add some logging to the finality verifier pallet

* Add finality target to happy path log
This commit is contained in:
Hernando Castano
2021-02-04 12:53:49 -05:00
committed by Bastian Köcher
parent 8c7d0ca35d
commit 51364e732c
1 changed files with 18 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
bridges/modules/finality-verifier/src/lib.rs
+18 -3
View File
@@ -104,6 +104,9 @@ pub mod pallet {
) -> DispatchResultWithPostInfo { ) -> DispatchResultWithPostInfo {
let _ = ensure_signed(origin)?; let _ = ensure_signed(origin)?;
frame_support::debug::trace!("Going to try and finalize header {:?}", finality_target);
frame_support::debug::trace!("Got ancestry proof of length {}", ancestry_proof.len());
ensure!( ensure!(
ancestry_proof.len() <= T::MaxHeadersInSingleProof::get().as_(), ancestry_proof.len() <= T::MaxHeadersInSingleProof::get().as_(),
<Error<T>>::OversizedAncestryProof <Error<T>>::OversizedAncestryProof
@@ -119,9 +122,14 @@ pub mod pallet {
voter_set, voter_set,
&justification, &justification,
) )
.map_err(|_| <Error<T>>::InvalidJustification)?; .map_err(|e| {
frame_support::debug::error!("Received invalid justification for {:?}: {:?}", finality_target, e);
<Error<T>>::InvalidJustification
})?;
let best_finalized = T::HeaderChain::best_finalized(); let best_finalized = T::HeaderChain::best_finalized();
frame_support::debug::trace!("Checking ancestry against best finalized header: {:?}", &best_finalized);
ensure!( ensure!(
T::AncestryChecker::are_ancestors(&best_finalized, &finality_target, &ancestry_proof), T::AncestryChecker::are_ancestors(&best_finalized, &finality_target, &ancestry_proof),
<Error<T>>::InvalidAncestryProof <Error<T>>::InvalidAncestryProof
@@ -129,8 +137,15 @@ pub mod pallet {
// Note that this won't work if we ever change the `ancestry_proof` format to be // Note that this won't work if we ever change the `ancestry_proof` format to be
// sparse since this expects a contiguous set of finalized headers. // sparse since this expects a contiguous set of finalized headers.
let _ = let _ = T::HeaderChain::append_finalized_chain(ancestry_proof).map_err(|_| {
T::HeaderChain::append_finalized_chain(ancestry_proof).map_err(|_| <Error<T>>::FailedToWriteHeader)?; frame_support::debug::error!("Failed to append finalized header chain.");
<Error<T>>::FailedToWriteHeader
})?;
frame_support::debug::info!(
"Succesfully imported finalized header chain for target header {:?}!",
finality_target
);
Ok(().into()) Ok(().into())
} }