Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Beta backports to 2.0.4 #9452

Merged
merged 6 commits into from
Sep 10, 2018
Merged

Beta backports to 2.0.4 #9452

merged 6 commits into from
Sep 10, 2018

Conversation

5chdn
Copy link
Contributor

@5chdn 5chdn commented Aug 31, 2018

@5chdn 5chdn added A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. A8-backport 🕸 Pull request is already reviewed well in another branch. M2-config 📂 Chain specifications and node configurations. labels Aug 31, 2018
@5chdn 5chdn added this to the 2.1 milestone Aug 31, 2018
niklasad1 and others added 3 commits September 6, 2018 11:39
… unify error handeling for light and full client (#9383)

* Provide the actual `account` for eth_coinbase

The previous implementation always provided the `zero address` on
`eth_coinbase` RPC. Now, instead the actual address is returned on
success or an error when no account(s) is found!

* full client `eth_coinbase` return err

In the full-client return an error when no account is found instead of
returning the `zero address`

* Remove needless blocks on single import

* Remove needless `static` lifetime on const

* Fix `rpc_eth_author` test
@5chdn 5chdn added A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Sep 8, 2018
sorpaas and others added 2 commits September 11, 2018 01:55
* Only check warp syncing for eth_getWorks

* Use SyncStatus::is_snapshot_syncing
* Close Blooms DB files before DB restoration

* PR Grumbles I

* PR Grumble

* Grumble
@5chdn 5chdn merged commit e2e1d22 into beta Sep 10, 2018
@5chdn 5chdn deleted the a5-backports-2004 branch September 10, 2018 20:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-pleasereview 🤓 Pull request needs code review. A8-backport 🕸 Pull request is already reviewed well in another branch. M2-config 📂 Chain specifications and node configurations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants