sigp / siren

User interface for Lighthouse

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

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:

  1. Under Logs, when no validators are present in the validator client, it shows:
    image
    while the vc logs:
    INFO No validators present msg: see `lighthouse account validator create --help` or the HTTP API documentation, service: notifier
    The No 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.

  1. The active validators shown under Validator Management seems to be the total validators, not active validators, as below (I have 2 active validators in my vc and 4 total validators)
    image

  2. 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 in withdrawal validator, or enforcing mandatory password setup at the startup.

  3. Some minor formatting issues under Validator Management, info for a particular validator:
    image

  4. I tried to withdraw a validator but it says:
    exit

  1. 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.

  2. handled in #197

  3. it was explained that siren defaults to the api-token to validate sensitive actions when session auth is not set.

  4. could not recreate, however will keep looking into

  5. handled in #198