Conversation
| SETUP_MODE: "" | ||
| EXTRA_OPTS: "" | ||
| EXISTING_PASSWORD: "" | ||
| NEW_PASSWORD: "" |
There was a problem hiding this comment.
This seems like required to be in the environment variables, because without this, if a user tries to change the configuration (e.g., add extra flag), they will be prompted to be required to enter this field (for new operator mode)
For import operator mode, the EXISTING_PASSWORD is already there as above
| if [ ! -f "$PASSWORD_FILE_PATH" ]; then | ||
| echo "$EXISTING_PASSWORD" > "$PASSWORD_FILE_PATH" | ||
| fi | ||
|
|
There was a problem hiding this comment.
I think doing this way is probably better? i.e., if a user mistakenly change the password when updating their config, that will not be effective. Because I have tested to allow changing password after the setup, and it just failed to decrypt the password and Anchor cannot start
|
Revise the backup option to only backup the keys and not other data (logs, database) in fc2e222. Thank you discord user @gbeast for the feedback: https://discord.com/channels/747647430450741309/880470045422153758/1488751232368775199 |
There was a problem hiding this comment.
Suggest required: true here. With required: false, a user can submit the
wizard with blank EXISTING_PASSWORD
And combined with the new [ ! -f "$PASSWORD_FILE_PATH" ] guard in entrypoint.sh, that locks in an empty password.txt and the only recovery is a full package reinstall
The Anchor dashboard is copied directly from: https://github.com/sigp/anchor/blob/stable/metrics/dashboards/anchor-dash.json