Skip to content

App Submission: NPM (Nginx Proxy Manager) #649

App Submission: NPM (Nginx Proxy Manager)

App Submission: NPM (Nginx Proxy Manager) #649

Triggered via pull request October 3, 2024 12:24
@nmfretznmfretz
synchronize #1296
Status Success
Total duration 16s
Artifacts

lint-apps.yml

on: pull_request_target
Lint apps
7s
Lint apps
Fit to window
Zoom out
Zoom in

Annotations

1 warning and 3 notices
"icon" and "gallery" needs to be empty for new app submissions: nginx-proxy-manager/umbrel-app.yml#L34
The "icon" and "gallery" fields must be empty for new app submissions as it is being created by the Umbrel team.
External port mapping "40080:80": nginx-proxy-manager/docker-compose.yml#L13
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.).
External port mapping "40443:443": nginx-proxy-manager/docker-compose.yml#L13
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.).
Potentially using unsafe user in service "docker-host": nginx-proxy-manager/docker-compose.yml#L1
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.