Skip to content

Commit

Permalink
Merge pull request #3547 from anoma/grarco/fix-fee-log-msg
Browse files Browse the repository at this point in the history
Improve fee tracing message
  • Loading branch information
mergify[bot] committed Jul 31, 2024
2 parents 29b711d + fb60b10 commit d8abc13
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
2 changes: 2 additions & 0 deletions .changelog/unreleased/improvements/3547-fix-fee-log-msg.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
- Improved tracing messages regarding MASP fee payment.
([\#3547](https://github.com/anoma/namada/pull/3547))
7 changes: 4 additions & 3 deletions crates/node/src/protocol.rs
Original file line number Diff line number Diff line change
Expand Up @@ -739,8 +739,8 @@ where
if !result.is_accepted() {
state.write_log_mut().drop_tx();
tracing::error!(
"The fee unshielding tx is invalid, some VPs rejected \
it: {:#?}",
"The first transaction in the batch failed to pay \
fees via the MASP, some VPs rejected it: {:#?}",
result.vps_result.rejected_vps
);
}
Expand Down Expand Up @@ -777,7 +777,8 @@ where
Err(e) => {
state.write_log_mut().drop_tx();
tracing::error!(
"The fee unshielding tx is invalid, wasm run failed: {}",
"The first transaction in the batch failed to pay fees \
via the MASP, wasm run failed: {}",
e
);
if let Error::GasError(_) = e {
Expand Down

0 comments on commit d8abc13

Please sign in to comment.