Skip to content

Commit

Permalink
fix: fixed security recommendation from codacy
Browse files Browse the repository at this point in the history
  • Loading branch information
creyD committed Nov 24, 2024
1 parent 256e2ad commit 6ab1eaf
Showing 1 changed file with 14 additions and 1 deletion.
15 changes: 14 additions & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
@@ -1,9 +1,15 @@
FROM python:3.12-slim
ARG VERSION=unkown
ARG VERSION=unknown

# Create a non-root user and group
RUN groupadd -r appuser && useradd -r -g appuser appuser

WORKDIR /app
COPY . .

# Change ownership of the application directory
RUN chown -R appuser:appuser /app

# Python setup
ENV PYTHONDONTWRITEBYTECODE=1
ENV PYTHONUNBUFFERED=1
Expand All @@ -14,11 +20,18 @@ ENV ENV=DEV
RUN pip install --no-cache-dir --upgrade -r requirements.txt
RUN pip install 'uvicorn[standard]'

# Switch to the non-root user
USER appuser

EXPOSE 9000
CMD ["uvicorn", "app.main:app", "--workers", "6" , "--host", "0.0.0.0", "--port", "9000"]

# Install curl
USER root
RUN apt-get update && apt-get install -y --no-install-recommends curl && apt-get clean

# Switch back to the non-root user
USER appuser

HEALTHCHECK --interval=30s --timeout=10s --retries=5 \
CMD curl --fail http://localhost:9000/openapi.json || exit 1

0 comments on commit 6ab1eaf

Please sign in to comment.