Add VPS deployment support with Caddy reverse proxy #483
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
malberts
force-pushed
the
add-vps-deployment
branch
7 times, most recently
from
f5c9b6e to
6164eed
Compare
malberts
changed the title
Collaborator
Author
|
Basic VPS setup works here. There is still some variable cleanup that can be done, but not worth spending more time on. |
Separate docker-compose files for VPS and CI use cases: - docker-compose.yml: VPS deployment with Caddy for automatic HTTPS - docker-compose.ci.yml: CI setup (renamed from existing) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
malberts
force-pushed
the
add-vps-deployment
branch
from
6164eed to
cd76426
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds VPS deployment support with a Caddy reverse proxy to enable production hosting of NeoWiki. The changes separate CI and production configurations, add automatic HTTPS support via Caddy, and improve data persistence with volume mounts for MediaWiki images.
Changes:
- Split docker-compose configuration into production (with Caddy) and CI-specific versions
- Added Caddy reverse proxy service with automatic HTTPS for production deployments
- Added comprehensive deployment documentation in README.md
- Removed anonymous edit permissions in the default configuration
- Added new Makefile targets for importing demo data and updating containers
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 10 comments.
Show a summary per file
| File | Description |
|---|---|
| Docker/docker-compose.yml | Removed port mappings, added Caddy service with reverse proxy, added volumes for persistence |
| Docker/docker-compose.ci.yml | New CI-specific compose file maintaining port mappings for testing |
| Docker/Caddyfile | New Caddy configuration for reverse proxy and automatic HTTPS |
| Docker/.env | Updated environment variables with production change markers, removed LOCAL_HOST/LOCAL_IP variables |
| Docker/README.md | New deployment documentation with setup instructions for VPS hosting |
| Docker/Makefile | Updated install-db to use MW_SERVER, added import-demo-data and update targets |
| Docker/SettingsTemplate.php | Disabled anonymous editing by default |
| .github/workflows/build-docker.yml | Updated CI workflow to use docker-compose.ci.yml for testing |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
JeroenDeDauw
added a commit
that referenced
this pull request
Jan 30, 2026
Without this, pulling a new image with DB schema changes would leave the wiki broken until update.php is manually run. Follows-up to #483 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Member
|
Had CC review this PR and it only came up with this "essential" change: |
Without this, pulling a new image with DB schema changes would leave the wiki broken until update.php is manually run. Follows-up to #483 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
For #482
Generated by Claude Code. Cleaned up and tweaked by be.