Skip to content

Commit c02274b

Browse files
committed
Remove duplicate cosmos event emission in EVM transaction processing
The change removes `ctx.EventManager().EmitEvents()` call that was duplicating events already emitted in the temporary context. This prevents double emission of the same events during EVM transaction execution. The events are already properly handled in the temporary context's commit phase.
1 parent 24758ef commit c02274b

File tree

3 files changed

+0
-4
lines changed

3 files changed

+0
-4
lines changed

.github/workflows/dependencies.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,3 @@ jobs:
2323
- name: "Dependency Review"
2424
uses: actions/dependency-review-action@v3
2525
if: env.GIT_DIFF
26-
- name: "Go vulnerability check"
27-
run: make vulncheck
28-
if: env.GIT_DIFF

x/evm/keeper/state_transition.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -234,7 +234,6 @@ func (k *Keeper) ApplyTransaction(ctx sdk.Context, tx *ethtypes.Transaction) (*t
234234
commit()
235235
// Since the post-processing can alter the log, we need to update the result
236236
res.Logs = types.NewLogsFromEth(receipt.Logs)
237-
ctx.EventManager().EmitEvents(tmpCtx.EventManager().Events())
238237
}
239238
}
240239

0 commit comments

Comments
 (0)