Add standard repo configuration and community files - #1
Conversation
…ines, and docker support Co-authored-by: Devanik21 <162272415+Devanik21@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces essential project infrastructure, including Docker configuration, GitHub templates for issues and pull requests, and documentation for contributions, security, and conduct. The review identifies several improvement opportunities: resolving a branch naming inconsistency between the security policy and contribution guide, updating the security policy to favor private reporting over public issues, and enhancing the Dockerfile security by implementing a non-root user.
| FROM python:3.9-slim | ||
|
|
||
| WORKDIR /app | ||
|
|
||
| COPY requirements.txt . | ||
|
|
||
| RUN pip install --no-cache-dir -r requirements.txt | ||
|
|
||
| COPY . . | ||
|
|
||
| EXPOSE 8501 | ||
|
|
||
| CMD ["streamlit", "run", "app.py", "--server.address=0.0.0.0"] |
There was a problem hiding this comment.
Running the container as the root user is a security risk. It is recommended to create a non-privileged user and use the USER instruction to switch to it.
FROM python:3.9-slim
RUN useradd -m appuser
WORKDIR /app
COPY --chown=appuser:appuser requirements.txt .
RUN pip install --no-cache-dir -r requirements.txt
COPY --chown=appuser:appuser . .
USER appuser
EXPOSE 8501
CMD ["streamlit", "run", "app.py", "--server.address=0.0.0.0"]
There was a problem hiding this comment.
FROM python:3.9-slim
RUN useradd -m appuser
WORKDIR /app
COPY --chown=appuser:appuser requirements.txt .
RUN pip install --no-cache-dir -r requirements.txt
COPY --chown=appuser:appuser . .
USER appuser
EXPOSE 8501
CMD ["streamlit", "run", "app.py", "--server.address=0.0.0.0"]
plz apply these changes jules
There was a problem hiding this comment.
Applied the changes to create and use a non-root user (appuser) for better security in the Docker image, as requested.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: Devanik21 <162272415+Devanik21@users.noreply.github.com>
1 participant
This commit adds several files to make the repository more complete and professional.
No original files were modified or deleted.
PR created automatically by Jules for task 2486335811855377259 started by @Devanik21