[Bug]: Media/Event Snapshot API ignores request params if event has ended

[leccelecce]

Checklist

• I have updated to the latest available Frigate version.
• I have cleared the cache of my browser.
• I have tried a different browser to see if it is related to my browser.
• I have tried reproducing the issue in incognito mode to rule out problems with any third party extensions or plugins I have installed.

Describe the problem you are having

The docs for the media event snapshot API state you can add query params for cropping the image/resizing to a certain height.

As far as I can see this is only actually implemented for live events. As soon as an event is completed (i.e. has an end time), Frigate just reads the snapshot from disk instead.

What I expect: query params are honoured regardless of whether the event is live or not.

Steps to reproduce

GET /api/events/<eventid>/snapshot.jpg?height=240&crop=1 on an event that has ended
Check the resulting image size

Version

0.15 and 0.16

In which browser(s) are you experiencing the issue with?

No response

Frigate config file

n/a

docker-compose file or Docker CLI command

n/a

Relevant Frigate log output

n/a

Relevant go2rtc log output

n/a

Operating system

Debian

Install method

Docker Compose

Network connection

Wired

Camera make and model

n/a

Screenshots of the Frigate UI's System metrics pages

n/a

Any other information that may be helpful

No response

[NickM-27]

this is not a bug and is working as expected, we can update the docs description to make it more clear what the behavior is

[NickM-27]

fixed in #19156

[leccelecce]

Understood, thanks!