Skip to content

fix: Docker security hardening#192

Closed
ChinmayShringi wants to merge 1 commit intomainfrom
fix/docker-hardening
Closed

fix: Docker security hardening#192
ChinmayShringi wants to merge 1 commit intomainfrom
fix/docker-hardening

Conversation

@ChinmayShringi
Copy link
Copy Markdown
Owner

Summary

  • Multi-stage Dockerfile: separate build and runtime stages
  • Run as non-root mirofish user
  • Serve frontend with serve static server instead of Vite dev server
  • Add HEALTHCHECK instruction to Dockerfile
  • Add healthcheck config to docker-compose.yml
  • Update .dockerignore with additional exclusions

Test plan

  • docker compose build succeeds
  • Container starts and healthcheck passes
  • Frontend is served correctly on port 3000
  • Backend API responds on port 5001

Original PR: 666ghj/MiroFish#232
Original Author: @0xNyk

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.

1 participant