Bump docker/login-action from 3.1.0 to 3.2.0 #181
docker-image.yml
on: pull_request
build-and-push-image
1m 32s
Annotations
10 warnings
[HIGH] Missing User Instruction:
Dockerfile#L53
A user should be specified in the dockerfile, otherwise the image will run as root
|
[MEDIUM] Apt Get Install Pin Version Not Defined:
Dockerfile#L36
When installing a package, its pin version should be defined
|
[MEDIUM] NPM Install Command Without Pinned Version:
Dockerfile#L36
Check if packages installed by npm are pinning a specific version.
|
[MEDIUM] Unpinned Actions Full Length Commit SHA:
.github/workflows/docker-image.yml#L47
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps mitigate the risk of a bad actor adding a backdoor to the action's repository, as they would need to generate a SHA-1 collision for a valid Git object payload. When selecting a SHA, you should verify it is from the action's repository and not a repository fork.
|
[MEDIUM] Unpinned Actions Full Length Commit SHA:
.github/workflows/docker-image.yml#L67
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps mitigate the risk of a bad actor adding a backdoor to the action's repository, as they would need to generate a SHA-1 collision for a valid Git object payload. When selecting a SHA, you should verify it is from the action's repository and not a repository fork.
|
[LOW] Chown Flag Exists:
Dockerfile#L13
It is considered a best practice for every executable in a container to be owned by the root user even if it is executed by a non-root user, only execution permissions are required on the file, not ownership
|
[LOW] Chown Flag Exists:
Dockerfile#L59
It is considered a best practice for every executable in a container to be owned by the root user even if it is executed by a non-root user, only execution permissions are required on the file, not ownership
|
[LOW] Chown Flag Exists:
Dockerfile#L60
It is considered a best practice for every executable in a container to be owned by the root user even if it is executed by a non-root user, only execution permissions are required on the file, not ownership
|
[LOW] Healthcheck Instruction Missing:
Dockerfile#L53
Ensure that HEALTHCHECK is being used. The HEALTHCHECK instruction tells Docker how to test a container to check that it is still working
|
[INFO] APT-GET Not Avoiding Additional Packages:
Dockerfile#L36
Check if any apt-get installs don't use '--no-install-recommends' flag to avoid installing additional packages.
|