Skip to content

backport downstream#25

Merged
5chdn merged 6 commits into
masterfrom
a5-backport-downstream
Nov 29, 2018
Merged

backport downstream#25
5chdn merged 6 commits into
masterfrom
a5-backport-downstream

Conversation

@5chdn
Copy link
Copy Markdown

@5chdn 5chdn commented Nov 29, 2018

No description provided.

tomusdrw and others added 6 commits November 28, 2018 11:30
* Refactor sync to add priority tasks.

* Send priority tasks notifications.

* Propagate blocks, optimize transactions.

* Implement transaction propagation. Use sync_channel.

* Tone down info.

* Prevent deadlock by not waiting forever for sync lock.

* Fix lock order.

* Don't use sync_channel to prevent deadlocks.

* Fix tests.
* closes openethereum#7009

adds parity_verifySignature RPC method

* removed unneccesary into

* adds support for chain replay protected signatures

* corrected chain replay protection check

* corrected possible overflow

* added tests

* use checked_sub to prevent possible overflow

* use saturating_mul to prevent possible overflow

* changed method signature to accept r,s,v components

* added unit tests

* more tests

* refactored tests for better readability
moved verify_signature tests to live in the same file.

* removed unneccesary imports

* fixed PR grumbles

* fixed rsv notation

* fixed rsv notation

* renamed BasicAcount to RecoveredAccount, Support zero chain_id

* fixed compile errors

* fixed tests

* doc comment
* Don't sync all peers after each response

* Update formating

* Fix tests: add `continue_sync` to `Sync_step`

* Update ethcore/sync/src/chain/mod.rs

Co-Authored-By: ngotchac <ngotchac@gmail.com>
* Calculate gas for deployment transaction

* Space fixed

* ethcore: style fix in public_creation_transaction
* Remove tendermint engine support

* Remove tendermint test json spec

* Fix ethcore test compile

* Remove tendermint test in sync
@5chdn 5chdn merged commit c9e651e into master Nov 29, 2018
@5chdn 5chdn deleted the a5-backport-downstream branch November 29, 2018 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants