Skip to content

Commit 36aabc3

Browse files
committed
eth/catalyst: simplify new payload param checks
1 parent 8e9107c commit 36aabc3

File tree

1 file changed

+6
-12
lines changed

1 file changed

+6
-12
lines changed

eth/catalyst/api.go

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -452,22 +452,16 @@ func (api *ConsensusAPI) NewPayloadV2(params engine.ExecutableData) (engine.Payl
452452
if params.Withdrawals == nil {
453453
return engine.PayloadStatusV1{Status: engine.INVALID}, engine.InvalidParams.With(errors.New("nil withdrawals post-shanghai"))
454454
}
455-
if params.ExcessBlobGas != nil {
456-
return engine.PayloadStatusV1{Status: engine.INVALID}, engine.InvalidParams.With(errors.New("non-nil excessBlobGas pre-cancun"))
457-
}
458-
if params.BlobGasUsed != nil {
459-
return engine.PayloadStatusV1{Status: engine.INVALID}, engine.InvalidParams.With(errors.New("non-nil params.BlobGasUsed pre-cancun"))
460-
}
461455
} else {
462456
if params.Withdrawals != nil {
463457
return engine.PayloadStatusV1{Status: engine.INVALID}, engine.InvalidParams.With(errors.New("non-nil withdrawals pre-shanghai"))
464458
}
465-
if params.ExcessBlobGas != nil {
466-
return engine.PayloadStatusV1{Status: engine.INVALID}, engine.InvalidParams.With(errors.New("non-nil excessBlobGas pre-cancun"))
467-
}
468-
if params.BlobGasUsed != nil {
469-
return engine.PayloadStatusV1{Status: engine.INVALID}, engine.InvalidParams.With(errors.New("non-nil params.BlobGasUsed pre-cancun"))
470-
}
459+
}
460+
if params.ExcessBlobGas != nil {
461+
return engine.PayloadStatusV1{Status: engine.INVALID}, engine.InvalidParams.With(errors.New("non-nil excessBlobGas pre-cancun"))
462+
}
463+
if params.BlobGasUsed != nil {
464+
return engine.PayloadStatusV1{Status: engine.INVALID}, engine.InvalidParams.With(errors.New("non-nil params.BlobGasUsed pre-cancun"))
471465
}
472466
return api.newPayload(params, nil, nil)
473467
}

0 commit comments

Comments
 (0)