Skip to content

Commit

Permalink
bugfix: attaching missing TxTracker.From to pending txs to store for …
Browse files Browse the repository at this point in the history
…forced batches.

Signed-off-by: Nikolay Nedkov <[email protected]>
  • Loading branch information
Psykepro committed Aug 3, 2023
1 parent 24cbf1a commit 494cde2
Showing 1 changed file with 9 additions and 7 deletions.
16 changes: 9 additions & 7 deletions sequencer/finalizer.go
Original file line number Diff line number Diff line change
Expand Up @@ -731,8 +731,16 @@ func (f *finalizer) handleForcedTxsProcessResp(ctx context.Context, request stat
}
}

from, err := state.GetSender(txResp.Tx)
if err != nil {
log.Errorf("handleForcedTxsProcessResp: failed to get sender: %s", err)
continue
}

processedTransaction := transactionToStore{
txTracker: nil,
txTracker: &TxTracker{
From: from,
},
response: txResp,
batchResponse: result,
batchNumber: request.BatchNumber,
Expand All @@ -743,12 +751,6 @@ func (f *finalizer) handleForcedTxsProcessResp(ctx context.Context, request stat
flushId: result.FlushID,
}

from, err := state.GetSender(txResp.Tx)
if err != nil {
log.Errorf("handleForcedTxsProcessResp: failed to get sender: %s", err)
continue
}

f.pendingTxsToStoreMux.Lock()
// global tracker
f.pendingTxsToStoreWG.Add(1)
Expand Down

0 comments on commit 494cde2

Please sign in to comment.