Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prysm VC ignores --wallet-dir tag and uses default path for auth-token file #14217

Closed
NeoPlays opened this issue Jul 15, 2024 · 2 comments
Closed
Labels
Bug Something isn't working Validator Client

Comments

@NeoPlays
Copy link

Describe the bug

Sine 5.0.4 the provided --wallet-dir flag is ignored by Prysm VC, which uses the default $HOME/.eth2validators/prysm-wallet-v2 value instead for the generation of the auth-token file.

Has this worked before in a previous version?

This used to work for 5.0.3

🔬 Minimal Reproduction

running in a docker container:

/app/cmd/validator/validator --accept-terms-of-use=true --web --holesky --datadir=/opt/app/data/db --wallet-dir=/opt/app/data/wallets --wallet-password-file=/opt/app/data/passwords/wallet-password

Error

`time="2024-07-15 08:21:56" level=info msg="Validator Client auth token for gRPC and REST authentication set at /.eth2validators/prysm-wallet-v2/auth-token" prefix=rpc`

Platform(s)

Linux (x86)

What version of Prysm are you running? (Which release)

5.0.4

Anything else relevant (validator index / public key)?

No response

@NeoPlays NeoPlays added the Bug Something isn't working label Jul 15, 2024
@james-prysm
Copy link
Contributor

--keymanager-token-file please use this flag for setting to a specified location now. this is just to be more consistent with other clients. here is some context ethereum/keymanager-APIs#74

the default of the token previously was the data-dir folder.

We will be investigating wallet dir consistency overall

@james-prysm
Copy link
Contributor

I'll close this issue for now, if anyone else runs into this please refer to the above, will continue working on wallet improvements but won't be included for a few more releases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Validator Client
Projects
None yet
Development

No branches or pull requests

2 participants