[Feat] Default webserver splash#1117
Merged
RichardAnderson merged 4 commits intoMay 23, 2026
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds a unified “Vito splash page” for unmanaged domains and updates both Nginx and Caddy installation flows to deploy it (and uninstall flows to clean it up), replacing prior default webserver behavior.
Changes:
- Added a shared HTML splash page and default vhost configs for Nginx and Caddy that serve it.
- Updated Nginx/Caddy installers to deploy the splash assets and default host configuration.
- Updated Nginx/Caddy uninstall scripts to remove
/var/www/vito-splash.
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| resources/views/ssh/services/webserver/vito-splash.blade.php | New shared HTML splash page written to /var/www/vito-splash/index.html. |
| resources/views/ssh/services/webserver/nginx/uninstall-nginx.blade.php | Removes splash directory on Nginx uninstall. |
| resources/views/ssh/services/webserver/nginx/default-vhost.blade.php | New Nginx default server block serving splash on :80/:443. |
| resources/views/ssh/services/webserver/nginx/default-ssl-vhost.blade.php | Removes old default SSL behavior (previously closed connections). |
| resources/views/ssh/services/webserver/caddy/uninstall-caddy.blade.php | Removes splash directory on Caddy uninstall. |
| resources/views/ssh/services/webserver/caddy/default-vhost.blade.php | New Caddy default site serving splash with tls internal. |
| app/Services/Webserver/Nginx.php | Adds deploySplash() and calls it during install. |
| app/Services/Webserver/Caddy.php | Adds deploySplash() and calls it during install. |
| app/Services/Webserver/AbstractWebserver.php | Introduces abstract deploySplash() API for webserver implementations. |
saeedvaziry
approved these changes
May 23, 2026
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
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.
This pull request introduces a new unified "splash page" for unmanaged domains on both Nginx and Caddy webservers, replacing the previous default configurations. It adds logic to deploy this splash page during installation and removes it during uninstallation, ensuring a consistent and branded experience for server owners and visitors. The implementation includes new deployment methods, updated templates, and cleanup procedures.