Closed nmfretz closed 2 months ago
Thank you for your submission! This is an automated linter that checks for common issues in pull requests to the Umbrel App Store.
Please review the linting results below and make any necessary changes to your submission.
Severity | File | Description |
---|---|---|
âšī¸ | frigate/docker-compose.yml |
External port mapping "8554:8554": Port mappings may be unnecessary for the app to function correctly. Docker's internal DNS resolves container names to IP addresses within the same network. External access to the web interface is handled by the app_proxy container. Port mappings are only needed if external access is required to a port not proxied by the app_proxy, or if an app needs to expose multiple ports for its functionality (e.g., DHCP, DNS, P2P, etc.). |
âšī¸ | frigate/docker-compose.yml |
External port mapping "8555:8555/tcp": Port mappings may be unnecessary for the app to function correctly. Docker's internal DNS resolves container names to IP addresses within the same network. External access to the web interface is handled by the app_proxy container. Port mappings are only needed if external access is required to a port not proxied by the app_proxy, or if an app needs to expose multiple ports for its functionality (e.g., DHCP, DNS, P2P, etc.). |
âšī¸ | frigate/docker-compose.yml |
External port mapping "8555:8555/udp": Port mappings may be unnecessary for the app to function correctly. Docker's internal DNS resolves container names to IP addresses within the same network. External access to the web interface is handled by the app_proxy container. Port mappings are only needed if external access is required to a port not proxied by the app_proxy, or if an app needs to expose multiple ports for its functionality (e.g., DHCP, DNS, P2P, etc.). |
âšī¸ | frigate/docker-compose.yml |
Potentially using unsafe user in service "web": The default container user "root" can lead to security vulnerabilities. If you are using the root user, please try to specify a different user (e.g. "1000:1000") in the compose file or try to set the UID/PUID and GID/PGID environment variables to 1000. |
Symbol | Description |
---|---|
â | Error: This must be resolved before this PR can be merged. |
â ī¸ | Warning: This is highly encouraged to be resolved, but is not strictly mandatory. |
âšī¸ | Info: This is just for your information. |
đ¤ This is an automated summary of installation tests:
â App installs successfully in umbrel-dev VM
â App UI returns a 200 status code
đ¸ UI screenshot:
This PR must still be reviewed and tested before merging.
Reviewed and tested
đ¤ This is an automated pull request to update frigate to version v0.14.1. This PR must be reviewed and tested before merging.