Skip to content

Docker security#48

Merged
douglowe merged 7 commits intodevelopfrom
docker-security
May 12, 2025
Merged

Docker security#48
douglowe merged 7 commits intodevelopfrom
docker-security

Conversation

@douglowe
Copy link
Copy Markdown
Collaborator

@douglowe douglowe commented May 11, 2025 •
edited
Loading

Changes:

  • non-root account for running the flask and celery servers
  • docker ignore for important files (such as git logs, environment variables, docker compose scripts)
  • explicitly only loading app source code, rather than full working directory

@douglowe douglowe marked this pull request as ready for review May 11, 2025 21:14
@douglowe douglowe requested a review from alexhambley May 11, 2025 21:14
@douglowe douglowe self-assigned this May 11, 2025
@douglowe douglowe requested a review from OliverWoolland May 11, 2025 22:50
OliverWoolland
OliverWoolland approved these changes May 12, 2025
View reviewed changes
Copy link
Copy Markdown

@OliverWoolland OliverWoolland left a comment

Choose a reason for hiding this comment

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

Looks fine to me!

@douglowe douglowe merged commit da2fca2 into develop May 12, 2025
@douglowe douglowe deleted the docker-security branch May 16, 2025 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@OliverWoolland OliverWoolland OliverWoolland approved these changes

@alexhambley alexhambley Awaiting requested review from alexhambley

Assignees

@douglowe douglowe

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@douglowe @OliverWoolland