Skip to content

AWS-ready#142

Merged
elifouts merged 10 commits intomainfrom
aws-ready-main
Mar 9, 2026
Merged

AWS-ready#142
elifouts merged 10 commits intomainfrom
aws-ready-main

Conversation

@elifouts
Copy link
Collaborator

@elifouts elifouts commented Mar 7, 2026

This pull request updates the project to use the new production domain devbits.app instead of the previous devbits.ddns.net, and adds support for platform integration and compliance (such as account deletion and app linking). It also improves backend deployment and static asset handling, and updates documentation and configuration for AWS and systemd-based deployments. The most important changes are summarized below:

Production Domain Migration:

  • Switched all references from devbits.ddns.net to devbits.app across environment files, frontend/backend scripts, CORS origins, and user-facing messages. This ensures all environments and deployments use the new production domain. [1] [2] [3] [4] [5] [6] [7] [8] [9]

Platform Compliance and Static Asset Support:

  • Added static files for platform integration: apple-app-site-association (iOS universal links), assetlinks.json (Android app links), and a public account-deletion.html page for user data deletion requests. These are now served directly by the backend. [1] [2] [3] [4] [5]

Backend Deployment and Configuration:

  • Added a sample systemd service unit for backend deployment and an improved .env.example for AWS/RDS setups, including CORS origins and admin secrets. [1] [2]
  • Updated Dockerfile to copy static assets and adjusted docker-compose.yml to remove the Nginx service, simplifying deployment (static assets are now served by the Go backend). [1] [2] [3]

Documentation Updates:

  • Updated README.md to reference new backend AWS deployment instructions.

Code Improvements:

  • Improved backend logic for URL scheme detection in uploads, and for resolving admin UI/static asset directories. [1] [2]

These changes collectively modernize the deployment, improve platform compatibility, and ensure regulatory compliance for user data handling.

Copilot AI review requested due to automatic review settings March 7, 2026 02:36
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Migrates the project’s production domain defaults to devbits.app and updates deployment to support an AWS/systemd “native Go binary” path with backend-served static compliance/app-link assets.

Changes:

  • Replaced devbits.ddns.net with devbits.app across frontend config, scripts, and backend CORS defaults.
  • Added native AWS deploy tooling (build script, systemd unit templating, deploy wrappers) and simplified Docker deployment by removing nginx.
  • Added/served static files needed for app linking and compliance (AASA, assetlinks, privacy policy, account deletion) directly from the Go backend.

Reviewed changes

Copilot reviewed 29 out of 29 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
frontend/services/api.ts Updates default API base URL to devbits.app and fallback logging.
frontend/public/privacy-policy.html Updates public policy link to new domain.
frontend/eas.json Updates production env URLs to devbits.app.
frontend/app/settings/about.tsx Updates default site URL to devbits.app.
frontend/app.json Updates iOS associated domains to devbits.app.
frontend/DEEP_LINK_SETUP.md Updates deep-link hosting instructions to new domain.
backend/scripts/update-live.sh Switches live update flow to native deploy wrapper.
backend/scripts/update-live.ps1 Switches Windows flow to SSH remote native deploy.
backend/scripts/install-aws-systemd-service.sh New systemd installer/templater for native deployment.
backend/scripts/deploy-aws-native.sh New “build + install/restart systemd” deploy script.
backend/scripts/build-backend-linux.sh New Linux build script producing bin/devbits-api.
backend/scripts/README.md Updates backend scripts documentation and AWS runbook pointer.
backend/nginx/nginx.conf Removes nginx runtime configuration (nginx no longer used).
backend/docs/AWS_TRANSFER_NO_NGINX.md Adds AWS deployment runbook for ALB + systemd setup.
backend/docker-compose.yml Removes nginx service and adds default CORS origins env.
backend/deploy/systemd/devbits-api.service Adds systemd unit template used by installer script.
backend/api/static/privacy-policy.html Adds backend-served privacy policy page.
backend/api/static/assetlinks.json Adds backend-served Android App Links manifest.
backend/api/static/apple-app-site-association Adds backend-served iOS Universal Links manifest.
backend/api/static/account-deletion.html Adds backend-served account deletion instructions page.
backend/api/main.go Updates allowed origins, admin dir resolution, serves static compliance/app-link files.
backend/api/internal/handlers/media_routes.go Uses X-Forwarded-Proto to build correct absolute upload URLs behind proxies.
backend/api/internal/handlers/media_ingest.go Updates example domain in comment.
backend/Dockerfile Copies api/static into runtime image.
backend/.env.example Adds AWS/RDS-focused env example and CORS defaults.
README.md Links new AWS deployment runbook.
Powershell-Scripts/run-front.ps1 Updates production backend selection text and env vars to devbits.app.
Bash-Scripts/run-front.sh Updates production backend selection text and env vars to devbits.app.
.env.example Updates default API URL to devbits.app.

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

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@elifouts
Copy link
Collaborator Author

elifouts commented Mar 7, 2026

@copilot open a new pull request to apply changes based on the comments in this thread

Copy link
Contributor

Copilot AI commented Mar 7, 2026

@elifouts I've opened a new pull request, #143, to work on those changes. Once the pull request is ready, I'll request review from you.

Copy link
Contributor

Copilot AI commented Mar 7, 2026

@grillinr I've opened a new pull request, #144, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI and others added 8 commits March 6, 2026 22:05
…SE_URL const mutation, installer guard, docs (#143)

* Initial plan

* Address PR review comments: AASA content-type, proxy header security, template check, API_BASE_URL fix, docs update

Co-authored-by: elifouts <116454864+elifouts@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: elifouts <116454864+elifouts@users.noreply.github.com>
…nc script (#144)

* Initial plan

* Add sync-static.sh to prevent static file drift; fix account-deletion.html email

Co-authored-by: grillinr <169214325+grillinr@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: grillinr <169214325+grillinr@users.noreply.github.com>
…y; update TypeScript configurations and enhance local development experience.
…change default user to ec2-user and enhance clarity in documentation.
@elifouts elifouts requested a review from Copilot March 9, 2026 06:03
Copy link
Contributor

Copilot AI left a comment

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 56 out of 67 changed files in this pull request and generated 5 comments.

Comments suppressed due to low confidence (2)

frontend/components/FadeInImage.tsx:1

  • handleLoadEnd no longer forwards the event argument to the caller’s onLoadEnd, which is a behavior change for any consumer relying on the load event payload. Keep the callback signature as (event) => { ...; onLoadEnd?.(event); } so callers still receive the event data.
    frontend/app/(tabs)/index.tsx:1
  • Casting transform to as any largely defeats the benefit of adding ViewStyle typing and can mask real type errors in the animated style. Prefer a narrower cast (e.g. to the specific transform type) or adjust the style typing so the transform array is accepted without any.

@elifouts elifouts merged commit d0be354 into main Mar 9, 2026
6 of 7 checks passed
@elifouts
Copy link
Collaborator Author

elifouts commented Mar 9, 2026

backend deployed to ec2 instance. Ball is in your court now.

image

@elifouts
Copy link
Collaborator Author

elifouts commented Mar 9, 2026

image

@elifouts
Copy link
Collaborator Author

elifouts commented Mar 9, 2026

i need to go to bed

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.

4 participants