Skip to content

Commit a23b1dc

Browse files
committed
fix(templates): Add name field and fix nextcloud compose config
Nextcloud template was missing the required top-level name field and used direct port mapping instead of the proxy network. Aligned with other templates: use expose, container_name, and proxy network for nginx reverse proxying. Signed-off-by: nfebe <fenn25.fn@gmail.com>
1 parent 7cd0f18 commit a23b1dc

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

templates/nextcloud/docker-compose.yml

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,11 @@
1+
name: ${NAME}
12
services:
23
app:
34
image: nextcloud:latest
5+
container_name: ${NAME}
46
restart: unless-stopped
5-
ports:
6-
- "${PORT:-8080}:80"
7+
expose:
8+
- "80"
79
volumes:
810
- ./data:/var/www/html/data
911
- ./config:/var/www/html/config
@@ -17,9 +19,11 @@ services:
1719
- NEXTCLOUD_ADMIN_PASSWORD=${ADMIN_PASSWORD:-admin}
1820
- NEXTCLOUD_TRUSTED_DOMAINS=${TRUSTED_DOMAINS:-localhost}
1921
networks:
20-
- default
22+
- proxy
2123
- database
2224

2325
networks:
26+
proxy:
27+
external: true
2428
database:
2529
external: true

0 commit comments

Comments
 (0)