Undo #6819 - make yParity and v match - Githubissues

hyperledger / besu

An enterprise-grade Java-based, Apache 2.0 licensed Ethereum client https://wiki.hyperledger.org/display/besu

https://www.hyperledger.org/projects/besu

Apache License 2.0

1.44k stars 765 forks source link

Undo #6819 - make yParity and v match #7139

Closed shemnon closed 1 month ago

shemnon commented 1 month ago

PR description

Undo PR #6819 - for 2030 and 1559 transactions both v and yParity will be provided, and they will be the same number.

Fixed Issue(s)

fixes #7135

Thanks for sending a pull request! Have you done the following?

[ ] Checked out our contribution guidelines?
[ ] Considered documentation and added the doc-change-required label to this PR if updates are required.
[ ] Considered the changelog and included an update if required.
[ ] For database changes (e.g. KeyValueSegmentIdentifier) considered compatibility and performed forwards and backwards compatibility tests

Locally, you can run these tests to catch failures early:

[ ] unit tests: ./gradlew build
[ ] acceptance tests: ./gradlew acceptanceTest
[ ] integration tests: ./gradlew integrationTest
[ ] reference tests: ./gradlew ethereum:referenceTests:referenceTests