Skip to content

Add production Docker image - #247

Open
amicus-merakisys wants to merge 1 commit into
instructure:masterfrom
amicus-merakisys:add-production-dockerfile
Open

Add production Docker image#247
amicus-merakisys wants to merge 1 commit into
instructure:masterfrom
amicus-merakisys:add-production-dockerfile

Conversation

@amicus-merakisys

@amicus-merakisys amicus-merakisys commented Jun 19, 2026

Copy link
Copy Markdown

Summary

  • replace the default Dockerfile with a multi-stage production image that builds the React app and serves the static output with nginx
  • add nginx config for SPA fallback routing, long-lived hashed asset caching, and no-cache service worker delivery
  • update docker compose and README commands for the production container port
  • add .dockerignore to keep the Docker context clean

Why this matters

The previous Dockerfile was optimized for local development: it installed dependencies, copied the source tree, and ran the React development server. That works for contributor workflows, but it is heavier and less predictable for deployment. A production image should compile once, ship only the generated static assets, and serve them from a small web server.

Making the main Dockerfile production-ready means the default Docker path now matches how this viewer should be deployed: smaller runtime surface, faster startup, static asset caching, and proper SPA fallback behavior.

Testing

  • NODE_OPTIONS=--openssl-legacy-provider npm run build
  • Could not run docker build in this environment because Docker daemon access is unavailable: permission denied connecting to /var/run/docker.sock

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@amicus-merakisys
amicus-merakisys force-pushed the add-production-dockerfile branch from 572a025 to 75a7daa Compare June 19, 2026 15:27
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