Skip to content

Commit

Permalink
fix: report addresses without checksum (#384)
Browse files Browse the repository at this point in the history
The checksum serialization is causing issues for data science pipelines.
Luckily the new `Address` type uses lowercase hex for debug formatting.
  • Loading branch information
Theodus authored Aug 28, 2023
1 parent cdd9fc0 commit 2bb3d22
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions graph-gateway/src/client_query.rs
Original file line number Diff line number Diff line change
Expand Up @@ -414,7 +414,7 @@ async fn handle_client_query_inner(
match &auth {
AuthToken::ApiKey(api_key) => tracing::info!(
target: reports::CLIENT_QUERY_TARGET,
user_address = %api_key.user_address,
user_address = ?api_key.user_address,
api_key = %api_key.key,
),
AuthToken::Ticket(payload, _) => tracing::info!(
Expand Down Expand Up @@ -691,7 +691,7 @@ async fn handle_client_query_inner(
let span = tracing::info_span!(
target: reports::INDEXER_QUERY_TARGET,
"indexer_query",
indexer = %selection.indexing.indexer,
indexer = ?selection.indexing.indexer,
);
tokio::spawn(
async move {
Expand Down Expand Up @@ -828,10 +828,7 @@ async fn handle_indexer_query_inner(

let allocation = Address::from_slice(&receipt[0..20]);

tracing::info!(
target: reports::INDEXER_QUERY_TARGET,
allocation = allocation.to_string(),
);
tracing::info!(target: reports::INDEXER_QUERY_TARGET, ?allocation);

let response = result?;
if response.status != StatusCode::OK.as_u16() {
Expand Down

0 comments on commit 2bb3d22

Please sign in to comment.