Some feedback on Siren
chong-he opened this issue · comments
Play around with Siren (installed using Docker from the unstable
branch 39bd11f and have some feedbacks:
- Under
Logs
, when no validators are present in the validator client, it shows:
while thevc
logs:
INFO No validators present msg: see `lighthouse account validator create --help` or the HTTP API documentation, service: notifier
TheNo validators present
should be displayed after INFO in the UI. When there are validator(s), the UI displays the correct info.
On a side note, I just thought that it will be nice to have a scrolling bar for the logs.
-
The active validators shown under
Validator Management
seems to be the total validators, not active validators, as below (I have 2 active validators in myvc
and 4 total validators)
-
When entering Siren, it asks for Authentication where users can set a password. This can be skipped. However, if skipped, during
withdraw validator
, it is mandatory to enter a password. The solution here is to either making entering password optional inwithdrawal validator
, or enforcing mandatory password setup at the startup. -
Some minor formatting issues under
Validator Management
, info for a particular validator:
-
Was explained that siren displays the log data by the keys its receives via the sse log api. This data was not present during given log.
-
handled in #197
-
it was explained that siren defaults to the api-token to validate sensitive actions when session auth is not set.
-
could not recreate, however will keep looking into
-
handled in #198