lower log level
This commit is contained in:
parent
0cf51d6776
commit
40548bec4f
@ -10,7 +10,6 @@ use crate::config::{BlockAndRpc, TxHashAndRpc, Web3ConnectionConfig};
|
|||||||
use crate::frontend::authorization::{Authorization, RequestMetadata};
|
use crate::frontend::authorization::{Authorization, RequestMetadata};
|
||||||
use crate::jsonrpc::{JsonRpcForwardedResponse, JsonRpcRequest};
|
use crate::jsonrpc::{JsonRpcForwardedResponse, JsonRpcRequest};
|
||||||
use crate::rpcs::transactions::TxStatus;
|
use crate::rpcs::transactions::TxStatus;
|
||||||
use anyhow::Context;
|
|
||||||
use arc_swap::ArcSwap;
|
use arc_swap::ArcSwap;
|
||||||
use counter::Counter;
|
use counter::Counter;
|
||||||
use derive_more::From;
|
use derive_more::From;
|
||||||
|
@ -221,15 +221,16 @@ impl OpenRequestHandle {
|
|||||||
// only save reverts for some types of calls
|
// only save reverts for some types of calls
|
||||||
// TODO: do something special for eth_sendRawTransaction too
|
// TODO: do something special for eth_sendRawTransaction too
|
||||||
let error_handler = if let RequestErrorHandler::SaveReverts = error_handler {
|
let error_handler = if let RequestErrorHandler::SaveReverts = error_handler {
|
||||||
|
// TODO: should all these be Trace or Debug or a mix?
|
||||||
if !["eth_call", "eth_estimateGas"].contains(&method) {
|
if !["eth_call", "eth_estimateGas"].contains(&method) {
|
||||||
// trace!(%method, "skipping save on revert");
|
// trace!(%method, "skipping save on revert");
|
||||||
RequestErrorHandler::DebugLevel
|
RequestErrorHandler::TraceLevel
|
||||||
} else if self.authorization.db_conn.is_some() {
|
} else if self.authorization.db_conn.is_some() {
|
||||||
let log_revert_chance = self.authorization.checks.log_revert_chance;
|
let log_revert_chance = self.authorization.checks.log_revert_chance;
|
||||||
|
|
||||||
if log_revert_chance == 0.0 {
|
if log_revert_chance == 0.0 {
|
||||||
// trace!(%method, "no chance. skipping save on revert");
|
// trace!(%method, "no chance. skipping save on revert");
|
||||||
RequestErrorHandler::DebugLevel
|
RequestErrorHandler::TraceLevel
|
||||||
} else if log_revert_chance == 1.0 {
|
} else if log_revert_chance == 1.0 {
|
||||||
// trace!(%method, "gaurenteed chance. SAVING on revert");
|
// trace!(%method, "gaurenteed chance. SAVING on revert");
|
||||||
error_handler
|
error_handler
|
||||||
@ -237,7 +238,7 @@ impl OpenRequestHandle {
|
|||||||
< log_revert_chance
|
< log_revert_chance
|
||||||
{
|
{
|
||||||
// trace!(%method, "missed chance. skipping save on revert");
|
// trace!(%method, "missed chance. skipping save on revert");
|
||||||
RequestErrorHandler::DebugLevel
|
RequestErrorHandler::TraceLevel
|
||||||
} else {
|
} else {
|
||||||
// trace!("Saving on revert");
|
// trace!("Saving on revert");
|
||||||
// TODO: is always logging at debug level fine?
|
// TODO: is always logging at debug level fine?
|
||||||
@ -245,7 +246,7 @@ impl OpenRequestHandle {
|
|||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
// trace!(%method, "no database. skipping save on revert");
|
// trace!(%method, "no database. skipping save on revert");
|
||||||
RequestErrorHandler::DebugLevel
|
RequestErrorHandler::TraceLevel
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
error_handler
|
error_handler
|
||||||
|
Loading…
Reference in New Issue
Block a user