Grin: owner_api_listen_port missing field in grin-wallet.toml

Created on 28 Jan 2019  路  3Comments  路  Source: mimblewimble/grin

Related #2475.

A new config item was added in #2475 -

owner_api_listen_port

We need to be able to safely handle the case where this is missing (i.e. all existing wallets).

Otherwise the wallet fails to startup successfully with a missing field error.

bug help wanted must-have

All 3 comments

Not sure if the solution here is to recreate the config file (and fill in any missing values), or to safely handle this value as optional.

@yeastplume What's the recommended fix here? I know you've worked on some reading/rewriting config files stuff.

Note: workaround is to add it manually to grin-wallet.toml -

owner_api_listen_port = 3420

for backwards compat it needs to be an Option in the internal struct, then have something set it to the default value if it鈥檚 missing when read

Was this page helpful?
0 / 5 - 0 ratings

Related issues

RiyaKarthi picture RiyaKarthi  路  3Comments

antiochp picture antiochp  路  3Comments

antiochp picture antiochp  路  6Comments

lehnberg picture lehnberg  路  5Comments

Makadu59 picture Makadu59  路  6Comments