Skip to content
This repository has been archived by the owner on Apr 5, 2024. It is now read-only.

Ethsigner fails to read empty password files and crashes #372

Closed
jframe opened this issue May 28, 2021 · 0 comments · Fixed by #386
Closed

Ethsigner fails to read empty password files and crashes #372

jframe opened this issue May 28, 2021 · 0 comments · Fixed by #386
Assignees
Labels
TeamGroot Identifies a given issue is assigned to the Groot Team

Comments

@jframe
Copy link
Contributor

jframe commented May 28, 2021

Hi,

This looks like a regression somewhere. When I attempt to pass in an empty password file with v 21.3.0 I get the following error:

jfernandes@falcon:~/workspace/quorum-performance/besu(main)$ docker logs 265aae1a50cc
Setting logging level to INFO
2021-05-18 02:13:39.304+00:00 | main | INFO  | SignerSubCommand | Version = ethsigner/v21.3.0/linux-x86_64/-na-openjdk64bitservervm-java-11
2021-05-18 02:13:39.464+00:00 | main | ERROR | PasswordFileUtil | Cannot read password from empty file: /opt/ethsigner/passwordfile
Failed to construct a signer from supplied arguments.
Cause: Cannot read password from empty file: /opt/ethsigner/passwordfile
Usage: ethsigner file-based-signer [-hV] [-k=<FILE>] [-p=<FILE>]
Sign transactions with a key stored in an encrypted V3 Keystore file.
  -h, --help              Show this help message and exit.
  -k, --key-file=<FILE>   The path to a file containing the key used to sign
                            transactions.
  -p, --password-file=<FILE>
                          The path to a file containing the password used to
                            decrypt the keyfile.
  -V, --version           Print version information and exit.

This works fine on v0.6.0 though, which just starts up fine and handles requests

For ref the password file is https://github.com/ConsenSys/quorum-performance/blob/main/besu/config/besu/passwords.txt

To reproduce:

  1. Change this line https://github.com/ConsenSys/quorum-performance/blob/main/besu/docker-compose.yml#L318 to 21.3.0
  2. Run ./run.sh
  3. Check logs, all other ethsigners on 0.6.0 are fine, except this one which would have crashed
@jframe jframe added the TeamGroot Identifies a given issue is assigned to the Groot Team label May 28, 2021
@siladu siladu self-assigned this Sep 14, 2021
siladu added a commit to siladu/signers that referenced this issue Sep 16, 2021
It's possible to create web3 wallets with no password so it seems reasonable to allow this case

Relates to Consensys/ethsigner#372
siladu added a commit to Consensys/signers that referenced this issue Sep 24, 2021
Allow empty password files to be read

It's possible to create web3 wallets with no password so it seems reasonable to allow this case

Relates to Consensys/ethsigner#372
siladu added a commit to siladu/ethsigner that referenced this issue Sep 24, 2021
siladu added a commit that referenced this issue Sep 27, 2021
Upgrade to signers 1.0.19 to allow empty password files to be read during startup

Includes vertx upgrade and transitive azure key vault upgrade with breaking change to SECP256K1 curve.

Replace SECP256K1 curve (TestKey) with P-256K curve (TestKey2) in acceptance tests.

Fixes #372
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
TeamGroot Identifies a given issue is assigned to the Groot Team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants