[Bug]: Latest Frame API sets invalid Content-Type #19553
-
Checklist
Describe the problem you are havinghttps://docs.frigate.video/integrations/api/latest-frame-camera-name-latest-extension-get
Steps to reproduce
Version0.16 In which browser(s) are you experiencing the issue with?No response Frigate config filen/a docker-compose file or Docker CLI commandn/a Relevant Frigate log outputn/a Relevant go2rtc log outputn/a Operating systemOther Linux Install methodDocker Compose Network connectionMixed Camera make and modeln/a Screenshots of the Frigate UI's System metrics pagesn/a Any other information that may be helpfulThe bug is most likely related to this: mobilityhouse/ocpp#447 frigate/frigate/api/defs/query/media_query_parameters.py Lines 8 to 12 in 856aab8 And Line 156 in 856aab8 And I've never touched FastAPI, and I might be wrong, but the docs says |
Beta Was this translation helpful? Give feedback.
Replies: 3 comments 5 replies
-
The Latest Frame API endpoint you're referring to is documented in the Frigate HTTP API (1)(2). The endpoint accepts various image extensions including webp, png, jpg, and jpeg (3)(4). Based on your bug report, you're experiencing an issue where the Content-Type header is incorrectly set to The API endpoint Since this is a bug report about incorrect Content-Type headers in the HTTP response, and the issue appears to be in the FastAPI implementation rather than the documented API specification, you should report this on the Frigate GitHub repository for the development team to address. 📚 Sources: Have another question? Just tag @inkeep. |
Beta Was this translation helpful? Give feedback.
-
Thanks, this is fixed in #19555 |
Beta Was this translation helpful? Give feedback.
-
I don't want to nitpick, but with Line 156 in 856aab8 removed, .jpg will result in an invalid MIME type image/jpg . It should be image/jpeg .Also, in Python 3.11 StrEnum was introduced, which behaves like class MyEnum(str, Enum) pre 3.11 when it comes to string formatting, if that comes in handy.
|
Beta Was this translation helpful? Give feedback.
Thanks, this is fixed in #19555