POC: Frigate HTTP API using FastAPI

[iursevla]

FastAPI POC

This is a POC of using FastAPI to replace Flask in Frigate HTTP API. It is using the defaults provided by FastAPI (uvicorn and starlette). This was discussed here. The intent is not to be merged right away but rather to further the discussion and to get your feedback on my proposals at the end of this description.

I'm proposing that we add the FastAPI app mount the existing Flask app as a sub-application + 2. This means that the entire application still works as is and we can then start moving endpoint by endpoint to FastAPI until we get to a point where all endpoints are converted. At that point, we can remove Flask altogether.

I've started the conversion of 2-3 endpoints to showcase how it works:

• Preview endpoints:
  • /preview/{camera_name}/start/{start_ts}/end/{end_ts}
  • /preview/{camera_name}/start/{start_ts}/end/{end_ts}/frames
• Media endpoints:
  • /{camera_name}/latest.{extension} -> See warning below
• App endpoints:
  • /logs/{service}

Here it is working with FastAPI

converted_file.mp4

---

HTTP API Documentation

Without any extra effort, the swagger documentation is available at http://localhost:5001/docs or http://localhost:5001/redoc See documentation

See image && video

converted_file.mp4

The JSON/YAML schema can then be used to feed the HTTP API page without any human interaction.

---

Problems faced

Caution

I've faced a problem with the existing endpoints which have an extension suffix: .(jpg|jpeg|png|webp|gif).As a result ⚠️ This lines in nginx.conf was commented out. I've left a comment explaining the errors. If someone wants/can help me figure out why it is failing, then use my branch to debug the endpoints that contain an extension.

---

Proposal(s):

My main proposal is (assuming we can get the nginx problem I've faced resolved):

• Create a feature branch from the current development and merge this POC code in there
• Then I can then start moving 2-3 endpoints a day/every 2 days to FastAPI and each time I create PR that can be reviewed and merged into the feature branch

Other proposals:

1. In the previous POC I was not specific about the types for each path/query parameter but we can be more precise with the types

Details

By being specific the endpoint will fail if invoked with invalid parameters

and we also get extra auto-complete on the docs

1. Add prefix to API endpoints e.g. media related endpoints should be /api/media/... this avoid any conflict between endpoints and makes it easier to read and more "RESTFUL".

   i.e., the following should all be `/api/media/...`

   

2. Split app.py file into multiple other route files:

   • Move the config endpoints to config.py
   • Move the timeline endpoints to timeline.py
   • Move the logs endpoints tologs.py
   • Move the app lifecycle endpoints (stats/version/restart/etc) to lifecycle.py

Proposals 2 and 3 probably should be left out for future implementation to avoid scope creep