Skip to content

Commit

Permalink
strip out istanbul extra
Browse files Browse the repository at this point in the history
  • Loading branch information
alecps committed Jul 9, 2024
1 parent 1f524a8 commit a85e7b6
Showing 1 changed file with 11 additions and 11 deletions.
22 changes: 11 additions & 11 deletions compat_tests/compat_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -277,35 +277,35 @@ func filterResponses(res1, res2 json.RawMessage) (json.RawMessage, json.RawMessa
return nil, nil, err
}

extraData1, err := execJQ([]byte(res1), ".extraData")
extraData1Hex, err := execJQ([]byte(res1), ".extraData")
if err != nil {
return nil, nil, err
}

extraData1HexString := strings.TrimSuffix(string(extraData1Hex), "\"\n")
extraData1HexString = strings.Trim(extraData1HexString, `"\"`)

Check failure on line 285 in compat_tests/compat_test.go

View workflow job for this annotation

GitHub Actions / Lint

SA1024: cutset contains duplicate characters (staticcheck)
extraData1 := hexutil.MustDecode(extraData1HexString)
if len(extraData1) < IstanbulExtraVanity {
return nil, nil, fmt.Errorf("invalid istanbul header extra-data length from res1: %d", len(extraData1))
}

istanbulExtra := IstanbulExtra{}
err = rlp.DecodeBytes(hexutil.Bytes(extraData1)[IstanbulExtraVanity:], &istanbulExtra)
err = rlp.DecodeBytes(extraData1[IstanbulExtraVanity:], &istanbulExtra)
if err != nil {
return nil, nil, err
}

// istanbulExtra := IstanbulExtra{}
// err = json.Unmarshal(.UnmarshalJSON).UnmarshalFixedJSON()Bytes(extraData1)[IstanbulExtraVanity:], &istanbulExtra)
// if err != nil {
// return nil, nil, err
// }

istanbulExtra.AggregatedSeal = IstanbulAggregatedSeal{}

payload, err := json.Marshal(&istanbulExtra)
payload, err := rlp.EncodeToBytes(&istanbulExtra)
if err != nil {
return nil, nil, err
}

res1Filtered, err = execJQ([]byte(res1Filtered), ".extraData |=", string(append(extraData1[:IstanbulExtraVanity], payload...)))
res1Filtered, err = execJQ([]byte(res1Filtered), `.extraData = "`+hexutil.Encode(append(extraData1[:IstanbulExtraVanity], payload...))+`"`)

Check failure on line 304 in compat_tests/compat_test.go

View workflow job for this annotation

GitHub Actions / Lint

unnecessary conversion (unconvert)

if err != nil {
return nil, nil, err
}

return res1Filtered, res2Filtered, nil
}
Expand Down

0 comments on commit a85e7b6

Please sign in to comment.