Skip to content

Commit 55ce041

Browse files
authored
fix(metadata_txn): use actual execution result for receipt success status (#287)
Previously the receipt success field was hardcoded to true. Now it correctly reflects the actual execution result via is_success().
1 parent 02a11d0 commit 55ce041

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

crates/pipe-exec-layer-ext-v2/execute/src/onchain_config/metadata_txn.rs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ impl SystemTxnResult {
123123
result: BlockExecutionResult {
124124
receipts: vec![Receipt {
125125
tx_type,
126-
success: true,
126+
success: self.result.is_success(),
127127
cumulative_gas_used: gas_used,
128128
logs: self.result.into_logs(),
129129
}],
@@ -170,11 +170,13 @@ impl SystemTxnResult {
170170
receipt.cumulative_gas_used += gas_used;
171171
}
172172

173+
let is_success = self.result.is_success();
174+
173175
result.execution_output.receipts.insert(
174176
insert_position,
175177
Receipt {
176178
tx_type: self.txn.tx_type(),
177-
success: true,
179+
success: is_success,
178180
cumulative_gas_used,
179181
logs: self.result.into_logs(),
180182
},

0 commit comments

Comments
 (0)