vikunja
/
vikunja
6
43
Fork 72
Code Issues 150 Pull Requests 19 Packages Releases 32 Activity

Added section to full-docker-example.md for Caddy v2. #595

Merged
konrad merged 3 commits from leggettc18/api:master into master 2020-06-22 04:48:02 +00:00
Conversation 2 Commits 3 Files Changed 1 +3 -10
1 changed files with 3 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit c6980a390f - Show all commits

13
docs/content/doc/setup/full-docker-example.md
View File

@ -260,22 +260,15 @@ services:
frontend:
image: vikunja/frontend
restart: unless-stopped
proxy:
image: nginx
ports:
- 80:80
volumes:
- ./nginx.conf:/etc/nginx/conf.d/default.conf:ro
depends_on:
- api
- frontend
restart: unless-stopped
caddy:
leggettc18 marked this conversation as resolved
konrad commented 2020-06-22 03:44:09 +00:00
Review
Copy Link

I guess you would not need the nginx proxy when using caddy? Could you remove that?

I guess you would not need the nginx proxy when using caddy? Could you remove that?
image: caddy
restart: unless-stopped
ports:
- "80:80"
- "443:443"
depends_on:
- api
- frontend
volumes:
- ./Caddyfile:/etc/caddy/Caddyfile:ro
konrad commented 2020-06-22 03:44:59 +00:00
Outdated
Review
Copy Link

Please also add a depends_on section like the proxy has.

Please also add a `depends_on` section like the proxy has.
{{< /highlight >}}