feat(#237): implement app_status_advanced endpoint

fusion44 / blitz_api

A management backend for the RaspiBlitz project written in Python / FastAPI

MIT License

18 stars 18 forks source link

Closed fusion44 closed 5 months ago

fusion44 commented 6 months ago

Some apps might give status information that is computationally to expensive to include in the normal status endpoint which can be polled more often.

cstenglein commented 6 months ago

Bugs:

Doesn't work if there is no appId specified (just loads forever).
Doesn't work with electrs. Request URL is /api/v1/apps/status/electrs Just returns this:

{
  "id": "electrs",
  "error": "appID not in list"
}

fusion44 commented 6 months ago

Changes:

fix for wrong function calls
returns { "initialSyncDone": 0 } or { "initialSyncDone": 1 } when called with electrs now
clarified in endpoint documentation that the endpoint isn't implemented for all backends
returns a 400 BAD REQUEST when appID is invalid

cstenglein commented 6 months ago

When calling the endpoint, I get the following result:


{
  "id": "electrs",
  "error": "script result processing error: /home/admin/config.scripts/bonus.electrs.sh status"
}

cstenglein commented 5 months ago

When is this merged?

So I can merge the webui functionality as well ;)