Skip to content

feat: route VIF production traffic#4

Merged
kaanyagci merged 1 commit into
mainfrom
codex/vif-production-routing
Jun 7, 2026
Merged

feat: route VIF production traffic#4
kaanyagci merged 1 commit into
mainfrom
codex/vif-production-routing

Conversation

@kaanyagci

@kaanyagci kaanyagci commented Jun 7, 2026

Copy link
Copy Markdown
Member

Summary

  • add vif.io nginx route to the shared production proxy
  • connect proxy to the VIF app overlay network
  • include VIF site template in deployment bundle
  • reject deployments configured with DEPLOY_SSH_USER=root

Deployment note

  • The deploy workflow now requires DEPLOY_SSH_USER to be a non-root SSH user.

Verification

  • branch was already pushed; deployment depends on production environment secrets

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR extends the shared production Nginx proxy stack to route traffic for vif.io by adding a dedicated Nginx site template, wiring the proxy service into the VIF application overlay network, and ensuring the new template is included in the deployment bundle.

Changes:

  • Added a new vif.io Nginx vhost template with HTTP→HTTPS redirect, ACME challenge handling, and HTTPS reverse proxying to the VIF upstream.
  • Connected the proxy to a new external production overlay network (vif_prod_app) for service discovery to the VIF upstream.
  • Updated the GitHub Actions deployment workflow to require and deploy the VIF network configuration and template.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
sites/vif-prod.conf.template Adds Nginx vhost config for vif.io (ACME + redirect + TLS reverse proxy).
envs/production/compose.yml Declares the external vif_prod_app overlay network for production deploys.
envs/production/.env.proxy Adds production VIF_* environment variables (server name, upstream, TLS paths).
compose.yml Bundles the new VIF site template config and attaches Nginx to vif_prod_app.
.github/workflows/manual-deploy.yml Includes VIF template in the deployment bundle and requires the VIF network secret.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Comment thread .github/workflows/manual-deploy.yml
@kaanyagci kaanyagci self-assigned this Jun 7, 2026
@kaanyagci kaanyagci marked this pull request as ready for review June 7, 2026 12:56
@kaanyagci kaanyagci merged commit 5522bd9 into main Jun 7, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants