Skip to content

Commit

Permalink
Revert "Use traces to extract the input for settlement::Observer (#… (
Browse files Browse the repository at this point in the history
#3237)

# Description
Reverts #3233

None of the Arbitrum nodes supports `trace_transaction`. Supposedly,
`arbtrace_transaction` should be used instead, but I couldn't make it
work.

Also, not all backup nodes (read alchemy) for other networks (base, gc)
support `trace_transaction`. I noticed that alchemy supports
`debug_traceTransaction` that could be used instead.

Overall, I should have checked this before merging the PR. I decided to
revert this PR since, right now, Arbitrum staging is unable to work
properly, and also, even with specific/per_chain implementation I am not
sure I will be able to make it work with all PRIMARY/BACKUP nodes on ALL
networks. This needs throughout testing first.
  • Loading branch information
sunce86 authored Jan 16, 2025
1 parent 2725d5c commit b4e2bbd
Show file tree
Hide file tree
Showing 5 changed files with 34 additions and 347 deletions.
11 changes: 2 additions & 9 deletions crates/autopilot/src/domain/eth/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -312,20 +312,15 @@ pub struct TradeEvent {
pub order_uid: domain::OrderUid,
}

/// A trace of a Call type of action.
#[derive(Debug, Clone, Default)]
pub struct TraceCall {
pub to: Address,
pub input: Calldata,
}

/// Any type of on-chain transaction.
#[derive(Debug, Clone, Default)]
pub struct Transaction {
/// The hash of the transaction.
pub hash: TxId,
/// The address of the sender of the transaction.
pub from: Address,
/// The call data of the transaction.
pub input: Calldata,
/// The block number of the block that contains the transaction.
pub block: BlockNo,
/// The timestamp of the block that contains the transaction.
Expand All @@ -334,6 +329,4 @@ pub struct Transaction {
pub gas: Gas,
/// The effective gas price of the transaction.
pub gas_price: EffectiveGasPrice,
/// Traces of all Calls contained in the transaction.
pub trace_calls: Vec<TraceCall>,
}
Loading

0 comments on commit b4e2bbd

Please sign in to comment.