-
-
Notifications
You must be signed in to change notification settings - Fork 106
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FR] Ability to override Frigate latest.jpg
and snapshot.jpg
#110
Comments
Looks like it will do what I need and more! Thanks! |
Awesome, thanks for the quick reply! I'll get this pushed up tonight and let you know when it's ready to test 🤞 |
It's awesome, it fits my needs to ! |
Hey @hawkeye217 and @Sylvain-Laine, this is ready for testing! The latest beta should include it Hopefully I didn't break anything haha! Let me know if it works and what you think. I updated the README to include the new options. |
So far it works ! |
Thanks for the fast turnaround! I'll install it and test it today. |
Tested the beta on my doorbell cam and another interior cam. It all seems to be working great for the doorbell cam, but the interior cam's snapshot url requires a username and password as follows:
I'm guessing the way you've set up axios doesn't allow for that, as I'm getting a 401 (unauthorized) error:
|
Hm...glad it's working at least for one of them. I haven't tried any URLs before that have username:password in them. Does going to |
Yes it does. The browser does basic authentication based on the username and password in the url. Axios requires a |
Ah, I didn't know the browser did that in the background. Ya it should be pretty easier to take that out of the URL then split it based on the await axios.get(url, {}, {
auth: {
username,
password
}
}); |
I tried to test this with Did you try that as well? The below worked and at least passed that URL check. When I remove events:
office:
image:
latest: http://foo:[email protected]/basic-auth/foo/bar |
Just FYI, |
Glad to hear! @hawkeye217 can you confirm if you were passing the |
# [1.1.0](v1.0.0...v1.1.0) (2021-09-25) ### Bug Fixes * call for new matches on paginated page after loading is set to false ([231ac12](231ac12)) * don't reset filters when all results on page are deleted ([#106](#106)) ([bf5ebac](bf5ebac)) * pass camera name when reprocessing image ([cbe7a57](cbe7a57)) * replace image-size with probe-image-size ([5b7816a](5b7816a)) ### Features * ability to override frigate options per camera ([#110](#110)) ([e2f93e6](e2f93e6)) * **ui:** camera and event type filters ([#106](#106)) ([c914308](c914308))
I decided not to use that specific camera with Double Take, so I didn't look into it any further. But if it works for others, I'm sure it was just something specific to my setup. Thanks! |
No problem. I'm going to close this since these changes got merged into the |
I am also getting 401 errors with
The "resolve" error does not make sense to me as the url is a IP not a domain name. I also tried with a .local domain but same error. The auth url is correct, working in a fresh incognito window. |
@miguelangel-nubla, that error just means the URL/IP did not respond within 1 second, so the request times out. I have a hard timeout of 1 second currently to prevent these status checks from hanging too long. Have you been able to verify the container can ping the IP? |
Yes connectivity is right. Upon further investigation the device asks for digest auth instead of basic auth. I made it work as a workaround with a reverse proxy that strips the digest auth and that works as expected, so the issue is definitely in the auth logic. |
Ah ok, good to know it's related to digest auth. If you want me to add in support for that, feel free to open a feature request. Glad you got it working with the reverse proxy for now. |
I have the similar config, but double take didn't get any picture after overriding the snapshot.
I test http://mycameraURL/snap.jpeg, and it works well in the browser. Here is the log
|
Continuing the discussion from #102 @hawkeye217 is looking for a way to override the
latest.jpg
andsnapshot.jpg
images rather than using the default ones from Frigate's API.Here is my proposed solution.
With this configuration the
office
camera has new values applied to it. I've disabled thelatest
processing by setting theattempts.latest
to0
. The MQTT snapshot is also disabled by settingattempts.mqtt
tofalse
. You can also override thesnapshot.jpg
orlatest.jpg
image instead of using the ones provided from Frigate's API.You can also change the
image.height
per camera, but this will only apply to the standardlatest.jpg
andsnapshot.jpg
images from Frigate. In the above example because I have a custom value forlatest
andsnapshot
, then the height will be omitted.@hawkeye217 I have all of these changes above ready locally. Just want to make sure the above makes sense before I release a new beta build with this. Let me know what you think!
The text was updated successfully, but these errors were encountered: