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

Contracts and accounts are disappeared #7632

Closed
ValeryDubrava opened this issue Jan 19, 2018 · 6 comments
Closed

Contracts and accounts are disappeared #7632

ValeryDubrava opened this issue Jan 19, 2018 · 6 comments
Assignees
Labels
F2-bug 🐞 The client fails to follow expected behavior. M4-core ⛓ Core client code / Rust. P2-asap 🌊 No need to stop dead in your tracks, however issue should be addressed as soon as possible.
Milestone

Comments

@ValeryDubrava
Copy link

ValeryDubrava commented Jan 19, 2018

Before filing a new issue, please provide the following information.

I'm running:

  • Which Parity version?: 1.8.6 beta-2d051e4-20180109
  • Which operating system?: Linux Debian 7
  • How installed?: via deb package
  • Are you fully synchronized?: yes
  • Which network are you connected to?: ropsten
  • Did you try to restart the node?: yes

Contracts and Accounts are disappeared after restart the node. In web3 there is all accounts (web3.eth.accounts = [{...},...]).
I've use ssh port forwarding: ssh -L 8545:127.0.0.1:8545 -L 8546:127.0.0.1:8546 -L 8180:127.0.0.1:8180 user@host
I've specified command line arguments: parity daemon ./parity.pid --db-compaction ssd --cache-size 1024 --force-ui --jsonrpc-cors all --identity myid --allow-ips public --log-file parity.log --nat extip:1.2.3.4 --chain ropsten --jsonrpc-apis all --pruning=archive --reserved-peers ropstenpeers.txt

Also, I've checked the directory with keys - all keys still there. It looks like some UI bug.

@ValeryDubrava
Copy link
Author

image
Screencast from the network tab of the dev tool.

@ValeryDubrava
Copy link
Author

ValeryDubrava commented Jan 19, 2018

Startup log:

2018-01-19 12:07:30  Starting Parity/v1.8.6-beta-2d051e4-20180109/x86_64-linux-gnu/rustc1.22.1
2018-01-19 12:07:30  Keys path /root/.local/share/io.parity.ethereum/keys/Ropsten
2018-01-19 12:07:30  DB path /root/.local/share/io.parity.ethereum/chains/test/db/ae90623718e47d66
2018-01-19 12:07:30  Path to dapps /root/.local/share/io.parity.ethereum/dapps
2018-01-19 12:07:30  State DB configuration: archive
2018-01-19 12:07:30  Operating mode: active
2018-01-19 12:07:30  Warning: Warp Sync is disabled because of non-default pruning mode.
2018-01-19 12:07:30  Configured for Ropsten using Ethash engine
2018-01-19 12:07:30  Updated conversion rate to Ξ1 = US$1058.32 (112487350 wei/gas)
2018-01-19 12:07:35  Public node URL: enode://<removed>:30303
2018-01-19 12:08:06     0/25 peers    715 KiB chain  222 KiB db  0 bytes queue 448 bytes sync  RPC:  2 conn,  4 req/s, 18489 µs
2018-01-19 12:08:21  Imported #2483679 0cef…df68 (6 txs, 0.22 Mgas, 628.58 ms, 1.40 KiB) + another 1 block(s) containing 26 tx(s)
2018-01-19 12:08:26  Error sending a subscription update: ConnectionClosed
2018-01-19 12:08:27  Cannot send notification: SendError("...")

@ValeryDubrava
Copy link
Author

ValeryDubrava commented Jan 19, 2018

Looks like I've found the right error:
image

@ValeryDubrava
Copy link
Author

I've found the root of the problem. File with key contains meta json as string. And that json was corrupted somehow... I do not know, how it happend. But it became corrupted after parity restart. Please, check it twice, because it looks like very critical issue, if parity corrupts key files on shutdown.

@5chdn 5chdn added F2-bug 🐞 The client fails to follow expected behavior. P5-sometimesoon 🌲 Issue is worth doing soon. M4-core ⛓ Core client code / Rust. labels Jan 22, 2018
@5chdn 5chdn modified the milestones: 1.9, 1.10 Jan 22, 2018
@butterstuff
Copy link

i have seen this issue as well - i wasnt sure it was a bug, so i just reimported a backed up key... but that is the same issue I had

@5chdn 5chdn added M7-ui and removed M4-core ⛓ Core client code / Rust. labels Jan 29, 2018
@beaugunderson
Copy link

Same issue here; diffed a backup of my keys to see which was the issue, here's the corruption in the "meta" key:

"meta":"{\"description\":\"\",\"passwordHi\",\"passwordHint\":\"\",\"tags\":[],\"timestamp\":1516865678141}"

I changed the name of this key recently.

@5chdn 5chdn added P2-asap 🌊 No need to stop dead in your tracks, however issue should be addressed as soon as possible. and removed P5-sometimesoon 🌲 Issue is worth doing soon. labels Feb 12, 2018
@tomusdrw tomusdrw self-assigned this Feb 12, 2018
@tomusdrw tomusdrw added M4-core ⛓ Core client code / Rust. and removed M7-ui labels Feb 12, 2018
tomusdrw added a commit that referenced this issue Feb 12, 2018
tomusdrw added a commit that referenced this issue Feb 12, 2018
@5chdn 5chdn closed this as completed in 90bd6bf Feb 12, 2018
tomusdrw added a commit that referenced this issue Feb 14, 2018
tomusdrw added a commit that referenced this issue Feb 14, 2018
This was referenced Feb 14, 2018
5chdn pushed a commit that referenced this issue Feb 14, 2018
* update back-references more aggressively after answering from cache (#7578)

* Add new EF ropstens nodes. (#7824)

* Add new EF ropstens nodes.

* Fix tests

* Add a timeout for light client sync requests (#7848)

* Add a timeout for light client sync requests

* Adjusting timeout to number of headers

* Flush keyfiles. Resolves #7632 (#7868)

* Fix wallet import (#7873)

* rpc: generate new account id for imported wallets

* ethstore: handle duplicate wallet filenames

* ethstore: simplify deduplication of wallet file names

* ethstore: do not dedup wallet filenames on update

* ethstore: fix minor grumbles

* [WASM] mem_cmp added to the Wasm runtime (#7539)

* mem_cmp added to the Wasm runtime

* schedule.wasm.mem_copy to schedule.wasm.mem_cmp for mem_cmp

* [Wasm] memcmp fix and test added (#7590)

* [Wasm] memcmp fix and test added

* [Wasm] use reqrep_test! macro for memcmp test

* wasmi interpreter (#7796)

* adjust storage update evm-style (#7812)

* disable internal memory (#7842)
5chdn pushed a commit that referenced this issue Feb 14, 2018
* update back-references more aggressively after answering from cache (#7578)

* Flush keyfiles. Resolves #7632 (#7868)

* Fix wallet import (#7873)

* rpc: generate new account id for imported wallets

* ethstore: handle duplicate wallet filenames

* ethstore: simplify deduplication of wallet file names

* ethstore: do not dedup wallet filenames on update

* ethstore: fix minor grumbles

* parity-version pr reopen (#7136)

* parity-version module split from util

removed unused util deps and features

trigger buildbot again

only kvdb links rocksdb

snappy linker issues

* rm snappy

* fixed old version imports

* Move updater metadata to Cargo.toml of parity-version. (#7832)

* Update version.

* Bump parity version.

* Fix version.

* Fix compilation.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
F2-bug 🐞 The client fails to follow expected behavior. M4-core ⛓ Core client code / Rust. P2-asap 🌊 No need to stop dead in your tracks, however issue should be addressed as soon as possible.
Projects
None yet
Development

No branches or pull requests

6 participants
@beaugunderson @tomusdrw @ValeryDubrava @5chdn @butterstuff and others