-
Notifications
You must be signed in to change notification settings - Fork 8
Frigate #394
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
base: master
Are you sure you want to change the base?
Conversation
@EnigmaCurry: this is working in that Frigate gets installed, the basics get configured, and you can log into it. I don't have any cameras to test video streaming so it's possible that I'll need to add something to |
frigate/Makefile
Outdated
@echo | ||
|
||
.PHONY: disable-tls # Disable TLS on Frigate's webserver | ||
disable-tls: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Normally in d.ry we setup a "config" container to setup any non env var enabled config. This would set the config file before the service starts. Can we do that instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is, of course, better way. Fixed in 10c27c6
For the extra ports needed, theres a few options: Add entrypoint through traefik. This has the benefit that you can bind to specific host ip address and apply an ip whitelist. But this has the overhead of passing large video frames through traefik. Add the port mapping directly in docker compose. We dont normally do this, but it might be preferred for constant video ingest.(no traefik overhead) however you lose the ip filter so you would want to be sure to bind it to a specific host ip address so its not publicly accessible. |
Implements #393