Skip to content

fix(docker): remove out-of-context plugins COPY, fix app entrypoint and host binding#442

Merged
utksh1 merged 2 commits into
utksh1:mainfrom
samarthsugandhi:fix/docker-build-context-and-entrypoint
May 31, 2026
Merged

fix(docker): remove out-of-context plugins COPY, fix app entrypoint and host binding#442
utksh1 merged 2 commits into
utksh1:mainfrom
samarthsugandhi:fix/docker-build-context-and-entrypoint

Conversation

@samarthsugandhi
Copy link
Copy Markdown
Contributor

Summary

Fixes three inconsistencies in the backend Dockerfile that prevented
docker compose up --build from working on a clean checkout.
Closes #381.

Changes

backend/Dockerfile

  • Removed COPY plugins ./plugins — the plugins/ directory is
    outside the ./backend build context so this line caused a build
    failure. Plugins are already mounted correctly via the compose
    volume ./plugins:/app/plugins, making the COPY redundant.
  • Fixed app entrypoint from secuscan.api:app to secuscan.main:app
    to match the actual FastAPI application module.
  • Fixed --host 127.0.0.1 to --host 0.0.0.0 so uvicorn binds to
    all interfaces inside the container and is reachable from the host
    via the mapped port.

How to test

  1. From the repository root run docker compose up --build
  2. Confirm the backend image builds without error
  3. Confirm the API is reachable at http://127.0.0.1:8081/api/v1/health

Notes

  • docker-compose.yml is unchanged — the compose file was already
    correct with context: ./backend and the plugins volume mount
  • No backend Python code changes

Closes #381

Copilot AI review requested due to automatic review settings May 31, 2026 06:59
Copy link
Copy Markdown

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@utksh1 utksh1 added level:intermediate 35 pts difficulty label for moderate contributor PRs type:devops DevOps or infrastructure work category bonus label type:bug Bug fix work category bonus label area:frontend Frontend React/UI work area:backend Backend API, database, or service work labels May 31, 2026
Copy link
Copy Markdown
Owner

@utksh1 utksh1 left a comment

Choose a reason for hiding this comment

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

Looks good: fixes the backend Dockerfile entrypoint/host binding and removes the invalid plugins COPY for the current compose build context; checks are green.

@utksh1 utksh1 added the gssoc:approved Admin validation: approved for GSSoC scoring label May 31, 2026
@utksh1 utksh1 merged commit e8697ff into utksh1:main May 31, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:backend Backend API, database, or service work area:frontend Frontend React/UI work gssoc:approved Admin validation: approved for GSSoC scoring level:intermediate 35 pts difficulty label for moderate contributor PRs type:bug Bug fix work category bonus label type:devops DevOps or infrastructure work category bonus label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Backend Dockerfile uses inconsistent build context and app module

3 participants