matous-volf/todo-baggins
matous-volf/todo-baggins
1
0
Fork 0
Code Issues 23 Pull Requests Actions Projects 1 Activity

feat: ability to create a project #9

Merged
matous-volf merged 50 commits from feat/project-create into main 2024-08-18 21:36:06 +00:00
Conversation 29 Commits 50 Files Changed 53 +1
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 5165d15d96 - Show all commits

1
docker/dev/app/Dockerfile
View File

@ -12,4 +12,5 @@ RUN npm install
coderabbitai[bot] commented 2024-08-16 22:39:20 +00:00 (Migrated from github.com)
Review
Copy Link

Consolidate RUN instructions for efficiency.

Multiple consecutive RUN instructions can be consolidated to reduce the number of layers in the Docker image.

RUN rustup target add wasm32-unknown-unknown && \
    cargo install dioxus-cli diesel_cli
Tools
Hadolint

[info] 4-4: Multiple consecutive RUN instructions. Consider consolidation.

(DL3059)

**Consolidate `RUN` instructions for efficiency.** Multiple consecutive `RUN` instructions can be consolidated to reduce the number of layers in the Docker image. ```dockerfile RUN rustup target add wasm32-unknown-unknown && \ cargo install dioxus-cli diesel_cli ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [info] 4-4: Multiple consecutive `RUN` instructions. Consider consolidation. (DL3059) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-16 22:39:20 +00:00 (Migrated from github.com)
Review
Copy Link

Pin versions in apt-get install.

It's recommended to pin versions in apt-get install to ensure consistent builds.

RUN apt-get update && apt-get install -y nodejs=12.22.12 npm=6.14.15 supervisor=4.2.2
Tools
Hadolint

[warning] 5-5: Pin versions in apt get install. Instead of apt-get install <package> use apt-get install <package>=<version>

(DL3008)

[info] 5-5: Delete the apt-get lists after installing something

(DL3009)

[info] 5-5: Avoid additional packages by specifying --no-install-recommends

(DL3015)

Clean up apt-get lists after installation.

To reduce the image size, delete the apt-get lists after installing packages.

RUN apt-get update && apt-get install -y --no-install-recommends nodejs npm supervisor && \
    rm -rf /var/lib/apt/lists/*
Tools
Hadolint

[warning] 5-5: Pin versions in apt get install. Instead of apt-get install <package> use apt-get install <package>=<version>

(DL3008)

[info] 5-5: Delete the apt-get lists after installing something

(DL3009)

[info] 5-5: Avoid additional packages by specifying --no-install-recommends

(DL3015)

Use --no-install-recommends to avoid unnecessary packages.

Using --no-install-recommends can help avoid installing additional packages that are not needed.

RUN apt-get update && apt-get install -y --no-install-recommends nodejs npm supervisor
Tools
Hadolint

[warning] 5-5: Pin versions in apt get install. Instead of apt-get install <package> use apt-get install <package>=<version>

(DL3008)

[info] 5-5: Delete the apt-get lists after installing something

(DL3009)

[info] 5-5: Avoid additional packages by specifying --no-install-recommends

(DL3015)

**Pin versions in `apt-get install`.** It's recommended to pin versions in `apt-get install` to ensure consistent builds. ```dockerfile RUN apt-get update && apt-get install -y nodejs=12.22.12 npm=6.14.15 supervisor=4.2.2 ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 5-5: Pin versions in apt get install. Instead of `apt-get install <package>` use `apt-get install <package>=<version>` (DL3008) --- [info] 5-5: Delete the apt-get lists after installing something (DL3009) --- [info] 5-5: Avoid additional packages by specifying `--no-install-recommends` (DL3015) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit --> --- **Clean up `apt-get` lists after installation.** To reduce the image size, delete the `apt-get` lists after installing packages. ```dockerfile RUN apt-get update && apt-get install -y --no-install-recommends nodejs npm supervisor && \ rm -rf /var/lib/apt/lists/* ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 5-5: Pin versions in apt get install. Instead of `apt-get install <package>` use `apt-get install <package>=<version>` (DL3008) --- [info] 5-5: Delete the apt-get lists after installing something (DL3009) --- [info] 5-5: Avoid additional packages by specifying `--no-install-recommends` (DL3015) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit --> --- **Use `--no-install-recommends` to avoid unnecessary packages.** Using `--no-install-recommends` can help avoid installing additional packages that are not needed. ```dockerfile RUN apt-get update && apt-get install -y --no-install-recommends nodejs npm supervisor ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 5-5: Pin versions in apt get install. Instead of `apt-get install <package>` use `apt-get install <package>=<version>` (DL3008) --- [info] 5-5: Delete the apt-get lists after installing something (DL3009) --- [info] 5-5: Avoid additional packages by specifying `--no-install-recommends` (DL3015) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-16 22:39:20 +00:00 (Migrated from github.com)
Review
Copy Link

Consolidate RUN instructions for efficiency.

Multiple consecutive RUN instructions can be consolidated to reduce the number of layers in the Docker image.

RUN rustup target add wasm32-unknown-unknown && \
    cargo install dioxus-cli diesel_cli
Tools
Hadolint

[info] 4-4: Multiple consecutive RUN instructions. Consider consolidation.

(DL3059)

**Consolidate `RUN` instructions for efficiency.** Multiple consecutive `RUN` instructions can be consolidated to reduce the number of layers in the Docker image. ```dockerfile RUN rustup target add wasm32-unknown-unknown && \ cargo install dioxus-cli diesel_cli ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [info] 4-4: Multiple consecutive `RUN` instructions. Consider consolidation. (DL3059) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-16 22:39:20 +00:00 (Migrated from github.com)
Review
Copy Link

Pin versions in apt-get install.

It's recommended to pin versions in apt-get install to ensure consistent builds.

RUN apt-get update && apt-get install -y nodejs=12.22.12 npm=6.14.15 supervisor=4.2.2
Tools
Hadolint

[warning] 5-5: Pin versions in apt get install. Instead of apt-get install <package> use apt-get install <package>=<version>

(DL3008)

[info] 5-5: Delete the apt-get lists after installing something

(DL3009)

[info] 5-5: Avoid additional packages by specifying --no-install-recommends

(DL3015)

Clean up apt-get lists after installation.

To reduce the image size, delete the apt-get lists after installing packages.

RUN apt-get update && apt-get install -y --no-install-recommends nodejs npm supervisor && \
    rm -rf /var/lib/apt/lists/*
Tools
Hadolint

[warning] 5-5: Pin versions in apt get install. Instead of apt-get install <package> use apt-get install <package>=<version>

(DL3008)

[info] 5-5: Delete the apt-get lists after installing something

(DL3009)

[info] 5-5: Avoid additional packages by specifying --no-install-recommends

(DL3015)

Use --no-install-recommends to avoid unnecessary packages.

Using --no-install-recommends can help avoid installing additional packages that are not needed.

RUN apt-get update && apt-get install -y --no-install-recommends nodejs npm supervisor
Tools
Hadolint

[warning] 5-5: Pin versions in apt get install. Instead of apt-get install <package> use apt-get install <package>=<version>

(DL3008)

[info] 5-5: Delete the apt-get lists after installing something

(DL3009)

[info] 5-5: Avoid additional packages by specifying --no-install-recommends

(DL3015)

**Pin versions in `apt-get install`.** It's recommended to pin versions in `apt-get install` to ensure consistent builds. ```dockerfile RUN apt-get update && apt-get install -y nodejs=12.22.12 npm=6.14.15 supervisor=4.2.2 ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 5-5: Pin versions in apt get install. Instead of `apt-get install <package>` use `apt-get install <package>=<version>` (DL3008) --- [info] 5-5: Delete the apt-get lists after installing something (DL3009) --- [info] 5-5: Avoid additional packages by specifying `--no-install-recommends` (DL3015) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit --> --- **Clean up `apt-get` lists after installation.** To reduce the image size, delete the `apt-get` lists after installing packages. ```dockerfile RUN apt-get update && apt-get install -y --no-install-recommends nodejs npm supervisor && \ rm -rf /var/lib/apt/lists/* ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 5-5: Pin versions in apt get install. Instead of `apt-get install <package>` use `apt-get install <package>=<version>` (DL3008) --- [info] 5-5: Delete the apt-get lists after installing something (DL3009) --- [info] 5-5: Avoid additional packages by specifying `--no-install-recommends` (DL3015) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit --> --- **Use `--no-install-recommends` to avoid unnecessary packages.** Using `--no-install-recommends` can help avoid installing additional packages that are not needed. ```dockerfile RUN apt-get update && apt-get install -y --no-install-recommends nodejs npm supervisor ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 5-5: Pin versions in apt get install. Instead of `apt-get install <package>` use `apt-get install <package>=<version>` (DL3008) --- [info] 5-5: Delete the apt-get lists after installing something (DL3009) --- [info] 5-5: Avoid additional packages by specifying `--no-install-recommends` (DL3015) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-17 10:13:37 +00:00 (Migrated from github.com)
Review
Copy Link

Avoid using USER root as the last instruction.

For security reasons, it's recommended to avoid running containers as the root user. Consider switching to a non-root user.

# Example: Create a non-root user and switch to it
RUN useradd -ms /bin/bash appuser
USER appuser
Tools
Hadolint

[warning] 15-15: Last USER should not be root

(DL3002)

**Avoid using `USER root` as the last instruction.** For security reasons, it's recommended to avoid running containers as the root user. Consider switching to a non-root user. ```dockerfile # Example: Create a non-root user and switch to it RUN useradd -ms /bin/bash appuser USER appuser ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 15-15: Last USER should not be root (DL3002) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-17 10:13:37 +00:00 (Migrated from github.com)
Review
Copy Link

Avoid using USER root as the last instruction.

For security reasons, it's recommended to avoid running containers as the root user. Consider switching to a non-root user.

# Example: Create a non-root user and switch to it
RUN useradd -ms /bin/bash appuser
USER appuser
Tools
Hadolint

[warning] 15-15: Last USER should not be root

(DL3002)

**Avoid using `USER root` as the last instruction.** For security reasons, it's recommended to avoid running containers as the root user. Consider switching to a non-root user. ```dockerfile # Example: Create a non-root user and switch to it RUN useradd -ms /bin/bash appuser USER appuser ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 15-15: Last USER should not be root (DL3002) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-17 19:37:36 +00:00 (Migrated from github.com)
Review
Copy Link

Consolidate RUN instructions for efficiency.

Consolidate the RUN instructions to reduce the number of layers in the Docker image.

RUN chown -R 1000:1000 /srv/app /usr/local/cargo /.local/share/dioxus && \
    mkdir -p /.local/share/dioxus
Tools
Hadolint

[info] 15-15: Multiple consecutive RUN instructions. Consider consolidation.

(DL3059)

[info] 16-16: Multiple consecutive RUN instructions. Consider consolidation.

(DL3059)

[info] 17-17: Multiple consecutive RUN instructions. Consider consolidation.

(DL3059)

**Consolidate `RUN` instructions for efficiency.** Consolidate the `RUN` instructions to reduce the number of layers in the Docker image. ```dockerfile RUN chown -R 1000:1000 /srv/app /usr/local/cargo /.local/share/dioxus && \ mkdir -p /.local/share/dioxus ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [info] 15-15: Multiple consecutive `RUN` instructions. Consider consolidation. (DL3059) --- [info] 16-16: Multiple consecutive `RUN` instructions. Consider consolidation. (DL3059) --- [info] 17-17: Multiple consecutive `RUN` instructions. Consider consolidation. (DL3059) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-17 19:37:36 +00:00 (Migrated from github.com)
Review
Copy Link

Consolidate RUN instructions for efficiency.

Consolidate the RUN instructions to reduce the number of layers in the Docker image.

RUN chown -R 1000:1000 /srv/app /usr/local/cargo /.local/share/dioxus && \
    mkdir -p /.local/share/dioxus
Tools
Hadolint

[info] 15-15: Multiple consecutive RUN instructions. Consider consolidation.

(DL3059)

[info] 16-16: Multiple consecutive RUN instructions. Consider consolidation.

(DL3059)

[info] 17-17: Multiple consecutive RUN instructions. Consider consolidation.

(DL3059)

**Consolidate `RUN` instructions for efficiency.** Consolidate the `RUN` instructions to reduce the number of layers in the Docker image. ```dockerfile RUN chown -R 1000:1000 /srv/app /usr/local/cargo /.local/share/dioxus && \ mkdir -p /.local/share/dioxus ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [info] 15-15: Multiple consecutive `RUN` instructions. Consider consolidation. (DL3059) --- [info] 16-16: Multiple consecutive `RUN` instructions. Consider consolidation. (DL3059) --- [info] 17-17: Multiple consecutive `RUN` instructions. Consider consolidation. (DL3059) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-18 20:56:28 +00:00 (Migrated from github.com)
Review
Copy Link

Use JSON notation for CMD instruction.

Using JSON notation for the CMD instruction ensures better handling of arguments and prevents issues related to shell interpretation.

- CMD sh docker/dev/app/entrypoint.sh
+ CMD ["sh", "docker/dev/app/entrypoint.sh"]
Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

CMD ["sh", "docker/dev/app/entrypoint.sh"]
Tools
Hadolint

[warning] 23-23: Use arguments JSON notation for CMD and ENTRYPOINT arguments

(DL3025)

**Use JSON notation for CMD instruction.** Using JSON notation for the CMD instruction ensures better handling of arguments and prevents issues related to shell interpretation. ```diff - CMD sh docker/dev/app/entrypoint.sh + CMD ["sh", "docker/dev/app/entrypoint.sh"] ``` <!-- suggestion_start --> <details> <summary>Committable suggestion</summary> > :bangbang: **IMPORTANT** > Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements. `````suggestion CMD ["sh", "docker/dev/app/entrypoint.sh"] ````` </details> <!-- suggestion_end --> <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 23-23: Use arguments JSON notation for CMD and ENTRYPOINT arguments (DL3025) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-18 20:56:28 +00:00 (Migrated from github.com)
Review
Copy Link

Use JSON notation for CMD instruction.

Using JSON notation for the CMD instruction ensures better handling of arguments and prevents issues related to shell interpretation.

- CMD sh docker/dev/app/entrypoint.sh
+ CMD ["sh", "docker/dev/app/entrypoint.sh"]
Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

CMD ["sh", "docker/dev/app/entrypoint.sh"]
Tools
Hadolint

[warning] 23-23: Use arguments JSON notation for CMD and ENTRYPOINT arguments

(DL3025)

**Use JSON notation for CMD instruction.** Using JSON notation for the CMD instruction ensures better handling of arguments and prevents issues related to shell interpretation. ```diff - CMD sh docker/dev/app/entrypoint.sh + CMD ["sh", "docker/dev/app/entrypoint.sh"] ``` <!-- suggestion_start --> <details> <summary>Committable suggestion</summary> > :bangbang: **IMPORTANT** > Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements. `````suggestion CMD ["sh", "docker/dev/app/entrypoint.sh"] ````` </details> <!-- suggestion_end --> <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 23-23: Use arguments JSON notation for CMD and ENTRYPOINT arguments (DL3025) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
COPY docker/dev/app/supervisord.conf /etc/supervisor/conf.d/supervisord.conf COPY docker/dev/app/supervisord.conf /etc/supervisor/conf.d/supervisord.conf
HEALTHCHECK CMD curl --fail http://localhost:8000 || exit 1 HEALTHCHECK CMD curl --fail http://localhost:8000 || exit 1
USER root
coderabbitai[bot] commented 2024-08-16 22:39:20 +00:00 (Migrated from github.com)
Review
Copy Link

Consolidate RUN instructions for efficiency.

Multiple consecutive RUN instructions can be consolidated to reduce the number of layers in the Docker image.

RUN rustup target add wasm32-unknown-unknown && \
    cargo install dioxus-cli diesel_cli
Tools
Hadolint

[info] 4-4: Multiple consecutive RUN instructions. Consider consolidation.

(DL3059)

**Consolidate `RUN` instructions for efficiency.** Multiple consecutive `RUN` instructions can be consolidated to reduce the number of layers in the Docker image. ```dockerfile RUN rustup target add wasm32-unknown-unknown && \ cargo install dioxus-cli diesel_cli ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [info] 4-4: Multiple consecutive `RUN` instructions. Consider consolidation. (DL3059) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-16 22:39:20 +00:00 (Migrated from github.com)
Review
Copy Link

Pin versions in apt-get install.

It's recommended to pin versions in apt-get install to ensure consistent builds.

RUN apt-get update && apt-get install -y nodejs=12.22.12 npm=6.14.15 supervisor=4.2.2
Tools
Hadolint

[warning] 5-5: Pin versions in apt get install. Instead of apt-get install <package> use apt-get install <package>=<version>

(DL3008)

[info] 5-5: Delete the apt-get lists after installing something

(DL3009)

[info] 5-5: Avoid additional packages by specifying --no-install-recommends

(DL3015)

Clean up apt-get lists after installation.

To reduce the image size, delete the apt-get lists after installing packages.

RUN apt-get update && apt-get install -y --no-install-recommends nodejs npm supervisor && \
    rm -rf /var/lib/apt/lists/*
Tools
Hadolint

[warning] 5-5: Pin versions in apt get install. Instead of apt-get install <package> use apt-get install <package>=<version>

(DL3008)

[info] 5-5: Delete the apt-get lists after installing something

(DL3009)

[info] 5-5: Avoid additional packages by specifying --no-install-recommends

(DL3015)

Use --no-install-recommends to avoid unnecessary packages.

Using --no-install-recommends can help avoid installing additional packages that are not needed.

RUN apt-get update && apt-get install -y --no-install-recommends nodejs npm supervisor
Tools
Hadolint

[warning] 5-5: Pin versions in apt get install. Instead of apt-get install <package> use apt-get install <package>=<version>

(DL3008)

[info] 5-5: Delete the apt-get lists after installing something

(DL3009)

[info] 5-5: Avoid additional packages by specifying --no-install-recommends

(DL3015)

**Pin versions in `apt-get install`.** It's recommended to pin versions in `apt-get install` to ensure consistent builds. ```dockerfile RUN apt-get update && apt-get install -y nodejs=12.22.12 npm=6.14.15 supervisor=4.2.2 ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 5-5: Pin versions in apt get install. Instead of `apt-get install <package>` use `apt-get install <package>=<version>` (DL3008) --- [info] 5-5: Delete the apt-get lists after installing something (DL3009) --- [info] 5-5: Avoid additional packages by specifying `--no-install-recommends` (DL3015) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit --> --- **Clean up `apt-get` lists after installation.** To reduce the image size, delete the `apt-get` lists after installing packages. ```dockerfile RUN apt-get update && apt-get install -y --no-install-recommends nodejs npm supervisor && \ rm -rf /var/lib/apt/lists/* ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 5-5: Pin versions in apt get install. Instead of `apt-get install <package>` use `apt-get install <package>=<version>` (DL3008) --- [info] 5-5: Delete the apt-get lists after installing something (DL3009) --- [info] 5-5: Avoid additional packages by specifying `--no-install-recommends` (DL3015) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit --> --- **Use `--no-install-recommends` to avoid unnecessary packages.** Using `--no-install-recommends` can help avoid installing additional packages that are not needed. ```dockerfile RUN apt-get update && apt-get install -y --no-install-recommends nodejs npm supervisor ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 5-5: Pin versions in apt get install. Instead of `apt-get install <package>` use `apt-get install <package>=<version>` (DL3008) --- [info] 5-5: Delete the apt-get lists after installing something (DL3009) --- [info] 5-5: Avoid additional packages by specifying `--no-install-recommends` (DL3015) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-17 10:13:37 +00:00 (Migrated from github.com)
Review
Copy Link

Avoid using USER root as the last instruction.

For security reasons, it's recommended to avoid running containers as the root user. Consider switching to a non-root user.

# Example: Create a non-root user and switch to it
RUN useradd -ms /bin/bash appuser
USER appuser
Tools
Hadolint

[warning] 15-15: Last USER should not be root

(DL3002)

**Avoid using `USER root` as the last instruction.** For security reasons, it's recommended to avoid running containers as the root user. Consider switching to a non-root user. ```dockerfile # Example: Create a non-root user and switch to it RUN useradd -ms /bin/bash appuser USER appuser ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 15-15: Last USER should not be root (DL3002) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-17 19:37:36 +00:00 (Migrated from github.com)
Review
Copy Link

Consolidate RUN instructions for efficiency.

Consolidate the RUN instructions to reduce the number of layers in the Docker image.

RUN chown -R 1000:1000 /srv/app /usr/local/cargo /.local/share/dioxus && \
    mkdir -p /.local/share/dioxus
Tools
Hadolint

[info] 15-15: Multiple consecutive RUN instructions. Consider consolidation.

(DL3059)

[info] 16-16: Multiple consecutive RUN instructions. Consider consolidation.

(DL3059)

[info] 17-17: Multiple consecutive RUN instructions. Consider consolidation.

(DL3059)

**Consolidate `RUN` instructions for efficiency.** Consolidate the `RUN` instructions to reduce the number of layers in the Docker image. ```dockerfile RUN chown -R 1000:1000 /srv/app /usr/local/cargo /.local/share/dioxus && \ mkdir -p /.local/share/dioxus ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [info] 15-15: Multiple consecutive `RUN` instructions. Consider consolidation. (DL3059) --- [info] 16-16: Multiple consecutive `RUN` instructions. Consider consolidation. (DL3059) --- [info] 17-17: Multiple consecutive `RUN` instructions. Consider consolidation. (DL3059) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-18 20:56:28 +00:00 (Migrated from github.com)
Review
Copy Link

Use JSON notation for CMD instruction.

Using JSON notation for the CMD instruction ensures better handling of arguments and prevents issues related to shell interpretation.

- CMD sh docker/dev/app/entrypoint.sh
+ CMD ["sh", "docker/dev/app/entrypoint.sh"]
Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

CMD ["sh", "docker/dev/app/entrypoint.sh"]
Tools
Hadolint

[warning] 23-23: Use arguments JSON notation for CMD and ENTRYPOINT arguments

(DL3025)

**Use JSON notation for CMD instruction.** Using JSON notation for the CMD instruction ensures better handling of arguments and prevents issues related to shell interpretation. ```diff - CMD sh docker/dev/app/entrypoint.sh + CMD ["sh", "docker/dev/app/entrypoint.sh"] ``` <!-- suggestion_start --> <details> <summary>Committable suggestion</summary> > :bangbang: **IMPORTANT** > Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements. `````suggestion CMD ["sh", "docker/dev/app/entrypoint.sh"] ````` </details> <!-- suggestion_end --> <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 23-23: Use arguments JSON notation for CMD and ENTRYPOINT arguments (DL3025) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
CMD ["supervisord", "-c", "/etc/supervisor/conf.d/supervisord.conf"] CMD ["supervisord", "-c", "/etc/supervisor/conf.d/supervisord.conf"]

coderabbitai[bot] commented 2024-08-16 22:39:20 +00:00 (Migrated from github.com)
Review
Copy Link

Consolidate RUN instructions for efficiency.

Multiple consecutive RUN instructions can be consolidated to reduce the number of layers in the Docker image.

RUN rustup target add wasm32-unknown-unknown && \
    cargo install dioxus-cli diesel_cli
Tools
Hadolint

[info] 4-4: Multiple consecutive RUN instructions. Consider consolidation.

(DL3059)

**Consolidate `RUN` instructions for efficiency.** Multiple consecutive `RUN` instructions can be consolidated to reduce the number of layers in the Docker image. ```dockerfile RUN rustup target add wasm32-unknown-unknown && \ cargo install dioxus-cli diesel_cli ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [info] 4-4: Multiple consecutive `RUN` instructions. Consider consolidation. (DL3059) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-16 22:39:20 +00:00 (Migrated from github.com)
Review
Copy Link

Pin versions in apt-get install.

It's recommended to pin versions in apt-get install to ensure consistent builds.

RUN apt-get update && apt-get install -y nodejs=12.22.12 npm=6.14.15 supervisor=4.2.2
Tools
Hadolint

[warning] 5-5: Pin versions in apt get install. Instead of apt-get install <package> use apt-get install <package>=<version>

(DL3008)

[info] 5-5: Delete the apt-get lists after installing something

(DL3009)

[info] 5-5: Avoid additional packages by specifying --no-install-recommends

(DL3015)

Clean up apt-get lists after installation.

To reduce the image size, delete the apt-get lists after installing packages.

RUN apt-get update && apt-get install -y --no-install-recommends nodejs npm supervisor && \
    rm -rf /var/lib/apt/lists/*
Tools
Hadolint

[warning] 5-5: Pin versions in apt get install. Instead of apt-get install <package> use apt-get install <package>=<version>

(DL3008)

[info] 5-5: Delete the apt-get lists after installing something

(DL3009)

[info] 5-5: Avoid additional packages by specifying --no-install-recommends

(DL3015)

Use --no-install-recommends to avoid unnecessary packages.

Using --no-install-recommends can help avoid installing additional packages that are not needed.

RUN apt-get update && apt-get install -y --no-install-recommends nodejs npm supervisor
Tools
Hadolint

[warning] 5-5: Pin versions in apt get install. Instead of apt-get install <package> use apt-get install <package>=<version>

(DL3008)

[info] 5-5: Delete the apt-get lists after installing something

(DL3009)

[info] 5-5: Avoid additional packages by specifying --no-install-recommends

(DL3015)

**Pin versions in `apt-get install`.** It's recommended to pin versions in `apt-get install` to ensure consistent builds. ```dockerfile RUN apt-get update && apt-get install -y nodejs=12.22.12 npm=6.14.15 supervisor=4.2.2 ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 5-5: Pin versions in apt get install. Instead of `apt-get install <package>` use `apt-get install <package>=<version>` (DL3008) --- [info] 5-5: Delete the apt-get lists after installing something (DL3009) --- [info] 5-5: Avoid additional packages by specifying `--no-install-recommends` (DL3015) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit --> --- **Clean up `apt-get` lists after installation.** To reduce the image size, delete the `apt-get` lists after installing packages. ```dockerfile RUN apt-get update && apt-get install -y --no-install-recommends nodejs npm supervisor && \ rm -rf /var/lib/apt/lists/* ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 5-5: Pin versions in apt get install. Instead of `apt-get install <package>` use `apt-get install <package>=<version>` (DL3008) --- [info] 5-5: Delete the apt-get lists after installing something (DL3009) --- [info] 5-5: Avoid additional packages by specifying `--no-install-recommends` (DL3015) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit --> --- **Use `--no-install-recommends` to avoid unnecessary packages.** Using `--no-install-recommends` can help avoid installing additional packages that are not needed. ```dockerfile RUN apt-get update && apt-get install -y --no-install-recommends nodejs npm supervisor ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 5-5: Pin versions in apt get install. Instead of `apt-get install <package>` use `apt-get install <package>=<version>` (DL3008) --- [info] 5-5: Delete the apt-get lists after installing something (DL3009) --- [info] 5-5: Avoid additional packages by specifying `--no-install-recommends` (DL3015) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-16 22:39:20 +00:00 (Migrated from github.com)
Review
Copy Link

Consolidate RUN instructions for efficiency.

Multiple consecutive RUN instructions can be consolidated to reduce the number of layers in the Docker image.

RUN rustup target add wasm32-unknown-unknown && \
    cargo install dioxus-cli diesel_cli
Tools
Hadolint

[info] 4-4: Multiple consecutive RUN instructions. Consider consolidation.

(DL3059)

**Consolidate `RUN` instructions for efficiency.** Multiple consecutive `RUN` instructions can be consolidated to reduce the number of layers in the Docker image. ```dockerfile RUN rustup target add wasm32-unknown-unknown && \ cargo install dioxus-cli diesel_cli ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [info] 4-4: Multiple consecutive `RUN` instructions. Consider consolidation. (DL3059) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-16 22:39:20 +00:00 (Migrated from github.com)
Review
Copy Link

Pin versions in apt-get install.

It's recommended to pin versions in apt-get install to ensure consistent builds.

RUN apt-get update && apt-get install -y nodejs=12.22.12 npm=6.14.15 supervisor=4.2.2
Tools
Hadolint

[warning] 5-5: Pin versions in apt get install. Instead of apt-get install <package> use apt-get install <package>=<version>

(DL3008)

[info] 5-5: Delete the apt-get lists after installing something

(DL3009)

[info] 5-5: Avoid additional packages by specifying --no-install-recommends

(DL3015)

Clean up apt-get lists after installation.

To reduce the image size, delete the apt-get lists after installing packages.

RUN apt-get update && apt-get install -y --no-install-recommends nodejs npm supervisor && \
    rm -rf /var/lib/apt/lists/*
Tools
Hadolint

[warning] 5-5: Pin versions in apt get install. Instead of apt-get install <package> use apt-get install <package>=<version>

(DL3008)

[info] 5-5: Delete the apt-get lists after installing something

(DL3009)

[info] 5-5: Avoid additional packages by specifying --no-install-recommends

(DL3015)

Use --no-install-recommends to avoid unnecessary packages.

Using --no-install-recommends can help avoid installing additional packages that are not needed.

RUN apt-get update && apt-get install -y --no-install-recommends nodejs npm supervisor
Tools
Hadolint

[warning] 5-5: Pin versions in apt get install. Instead of apt-get install <package> use apt-get install <package>=<version>

(DL3008)

[info] 5-5: Delete the apt-get lists after installing something

(DL3009)

[info] 5-5: Avoid additional packages by specifying --no-install-recommends

(DL3015)

**Pin versions in `apt-get install`.** It's recommended to pin versions in `apt-get install` to ensure consistent builds. ```dockerfile RUN apt-get update && apt-get install -y nodejs=12.22.12 npm=6.14.15 supervisor=4.2.2 ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 5-5: Pin versions in apt get install. Instead of `apt-get install <package>` use `apt-get install <package>=<version>` (DL3008) --- [info] 5-5: Delete the apt-get lists after installing something (DL3009) --- [info] 5-5: Avoid additional packages by specifying `--no-install-recommends` (DL3015) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit --> --- **Clean up `apt-get` lists after installation.** To reduce the image size, delete the `apt-get` lists after installing packages. ```dockerfile RUN apt-get update && apt-get install -y --no-install-recommends nodejs npm supervisor && \ rm -rf /var/lib/apt/lists/* ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 5-5: Pin versions in apt get install. Instead of `apt-get install <package>` use `apt-get install <package>=<version>` (DL3008) --- [info] 5-5: Delete the apt-get lists after installing something (DL3009) --- [info] 5-5: Avoid additional packages by specifying `--no-install-recommends` (DL3015) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit --> --- **Use `--no-install-recommends` to avoid unnecessary packages.** Using `--no-install-recommends` can help avoid installing additional packages that are not needed. ```dockerfile RUN apt-get update && apt-get install -y --no-install-recommends nodejs npm supervisor ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 5-5: Pin versions in apt get install. Instead of `apt-get install <package>` use `apt-get install <package>=<version>` (DL3008) --- [info] 5-5: Delete the apt-get lists after installing something (DL3009) --- [info] 5-5: Avoid additional packages by specifying `--no-install-recommends` (DL3015) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-17 10:13:37 +00:00 (Migrated from github.com)
Review
Copy Link

Avoid using USER root as the last instruction.

For security reasons, it's recommended to avoid running containers as the root user. Consider switching to a non-root user.

# Example: Create a non-root user and switch to it
RUN useradd -ms /bin/bash appuser
USER appuser
Tools
Hadolint

[warning] 15-15: Last USER should not be root

(DL3002)

**Avoid using `USER root` as the last instruction.** For security reasons, it's recommended to avoid running containers as the root user. Consider switching to a non-root user. ```dockerfile # Example: Create a non-root user and switch to it RUN useradd -ms /bin/bash appuser USER appuser ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 15-15: Last USER should not be root (DL3002) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-17 10:13:37 +00:00 (Migrated from github.com)
Review
Copy Link

Avoid using USER root as the last instruction.

For security reasons, it's recommended to avoid running containers as the root user. Consider switching to a non-root user.

# Example: Create a non-root user and switch to it
RUN useradd -ms /bin/bash appuser
USER appuser
Tools
Hadolint

[warning] 15-15: Last USER should not be root

(DL3002)

**Avoid using `USER root` as the last instruction.** For security reasons, it's recommended to avoid running containers as the root user. Consider switching to a non-root user. ```dockerfile # Example: Create a non-root user and switch to it RUN useradd -ms /bin/bash appuser USER appuser ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 15-15: Last USER should not be root (DL3002) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-17 19:37:36 +00:00 (Migrated from github.com)
Review
Copy Link

Consolidate RUN instructions for efficiency.

Consolidate the RUN instructions to reduce the number of layers in the Docker image.

RUN chown -R 1000:1000 /srv/app /usr/local/cargo /.local/share/dioxus && \
    mkdir -p /.local/share/dioxus
Tools
Hadolint

[info] 15-15: Multiple consecutive RUN instructions. Consider consolidation.

(DL3059)

[info] 16-16: Multiple consecutive RUN instructions. Consider consolidation.

(DL3059)

[info] 17-17: Multiple consecutive RUN instructions. Consider consolidation.

(DL3059)

**Consolidate `RUN` instructions for efficiency.** Consolidate the `RUN` instructions to reduce the number of layers in the Docker image. ```dockerfile RUN chown -R 1000:1000 /srv/app /usr/local/cargo /.local/share/dioxus && \ mkdir -p /.local/share/dioxus ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [info] 15-15: Multiple consecutive `RUN` instructions. Consider consolidation. (DL3059) --- [info] 16-16: Multiple consecutive `RUN` instructions. Consider consolidation. (DL3059) --- [info] 17-17: Multiple consecutive `RUN` instructions. Consider consolidation. (DL3059) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-17 19:37:36 +00:00 (Migrated from github.com)
Review
Copy Link

Consolidate RUN instructions for efficiency.

Consolidate the RUN instructions to reduce the number of layers in the Docker image.

RUN chown -R 1000:1000 /srv/app /usr/local/cargo /.local/share/dioxus && \
    mkdir -p /.local/share/dioxus
Tools
Hadolint

[info] 15-15: Multiple consecutive RUN instructions. Consider consolidation.

(DL3059)

[info] 16-16: Multiple consecutive RUN instructions. Consider consolidation.

(DL3059)

[info] 17-17: Multiple consecutive RUN instructions. Consider consolidation.

(DL3059)

**Consolidate `RUN` instructions for efficiency.** Consolidate the `RUN` instructions to reduce the number of layers in the Docker image. ```dockerfile RUN chown -R 1000:1000 /srv/app /usr/local/cargo /.local/share/dioxus && \ mkdir -p /.local/share/dioxus ``` <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [info] 15-15: Multiple consecutive `RUN` instructions. Consider consolidation. (DL3059) --- [info] 16-16: Multiple consecutive `RUN` instructions. Consider consolidation. (DL3059) --- [info] 17-17: Multiple consecutive `RUN` instructions. Consider consolidation. (DL3059) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-18 20:56:28 +00:00 (Migrated from github.com)
Review
Copy Link

Use JSON notation for CMD instruction.

Using JSON notation for the CMD instruction ensures better handling of arguments and prevents issues related to shell interpretation.

- CMD sh docker/dev/app/entrypoint.sh
+ CMD ["sh", "docker/dev/app/entrypoint.sh"]
Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

CMD ["sh", "docker/dev/app/entrypoint.sh"]
Tools
Hadolint

[warning] 23-23: Use arguments JSON notation for CMD and ENTRYPOINT arguments

(DL3025)

**Use JSON notation for CMD instruction.** Using JSON notation for the CMD instruction ensures better handling of arguments and prevents issues related to shell interpretation. ```diff - CMD sh docker/dev/app/entrypoint.sh + CMD ["sh", "docker/dev/app/entrypoint.sh"] ``` <!-- suggestion_start --> <details> <summary>Committable suggestion</summary> > :bangbang: **IMPORTANT** > Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements. `````suggestion CMD ["sh", "docker/dev/app/entrypoint.sh"] ````` </details> <!-- suggestion_end --> <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 23-23: Use arguments JSON notation for CMD and ENTRYPOINT arguments (DL3025) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->
coderabbitai[bot] commented 2024-08-18 20:56:28 +00:00 (Migrated from github.com)
Review
Copy Link

Use JSON notation for CMD instruction.

Using JSON notation for the CMD instruction ensures better handling of arguments and prevents issues related to shell interpretation.

- CMD sh docker/dev/app/entrypoint.sh
+ CMD ["sh", "docker/dev/app/entrypoint.sh"]
Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

CMD ["sh", "docker/dev/app/entrypoint.sh"]
Tools
Hadolint

[warning] 23-23: Use arguments JSON notation for CMD and ENTRYPOINT arguments

(DL3025)

**Use JSON notation for CMD instruction.** Using JSON notation for the CMD instruction ensures better handling of arguments and prevents issues related to shell interpretation. ```diff - CMD sh docker/dev/app/entrypoint.sh + CMD ["sh", "docker/dev/app/entrypoint.sh"] ``` <!-- suggestion_start --> <details> <summary>Committable suggestion</summary> > :bangbang: **IMPORTANT** > Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements. `````suggestion CMD ["sh", "docker/dev/app/entrypoint.sh"] ````` </details> <!-- suggestion_end --> <details> <summary>Tools</summary> <details> <summary>Hadolint</summary><blockquote> [warning] 23-23: Use arguments JSON notation for CMD and ENTRYPOINT arguments (DL3025) </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit -->