omni / poa-bridge

POA <-> Ethereum bridge for self transfers of POA native token to POA20 (ERC20 representation). Not supported. Use TokenBridge instead
https://github.com/poanetwork/token-bridge
GNU General Public License v3.0
79 stars 38 forks source link

a warning message if the keystore contains non-JSON files #88

Closed akolotov closed 6 years ago

akolotov commented 6 years ago

If the file that contains key for the bridge validator contains a non-JSON file a warning message appears:

INFO:bridge: Parsing cli arguments
INFO:bridge: Loading config
INFO:bridge: Starting event loop
INFO:bridge: Home rpc host http://127.0.0.1
INFO:bridge: Foreign rpc host http://127.0.0.1
INFO:bridge: Establishing connection:
INFO:bridge:   using RPC connection
WARN:ethstore::accounts_dir::disk: Invalid key file: "./keys/d6d713a71eb3955c680d5967e9bb22298edefcc3.pwd" (Error("invalid type: integer `11`, expected a valid key object", line: 2, column: 0))
INFO:bridge: Acquiring home & foreign chain ids
INFO:bridge: Home chain ID: 64206 Foreign chain ID: 45068
INFO:bridge: Reading the database
INFO:bridge: Loaded database
INFO:bridge: Starting listening to events

It is confusing since there is no strict understanding if it will impact bridge functionality or not.

yrashk commented 6 years ago

Short of extracting and forking some parts of ethstore it doesn't look we can do anything about this warning.