Dockerfilelint is an npm module that analyzes a Dockerfile and looks for common traps, mistakes and helps enforce best practices:

Start unit tests with npm test

./bin/dockerfilelint <path/to/Dockerfile>

You can configure the linter by creating a .dockerfilelintrc with the following syntax:

rules:
  uppercase_commands: off

The keys for the rules can be any file in the /lib/reference.js file. At this time, it's only possible to disable rules. They are all enabled by default.

The following rules are supported:

required_params
uppercase_commands
from_first
invalid_line
sudo_usage
apt-get_missing_param
apt-get_recommends
apt-get-upgrade
apt-get-dist-upgrade
apt-get-update_require_install
apkadd-missing_nocache_or_updaterm
apkadd-missing-virtual
invalid_port
invalid_command
expose_host_port
label_invalid
missing_tag
latest_tag
extra_args
missing_args
add_src_invalid
add_dest_invalid
invalid_workdir
invalid_format
apt-get_missing_rm

(Replace the pwd/Dockerfile with the path to your local Dockerfile)

sudo docker run -v `pwd`/Dockerfile:/Dockerfile dockerfilelint /Dockerfile

If you don't want to install this locally you can try it out on https://fromlatest.io.

• [x] This should be the first command in the Dockerfile
• [x] Base image should specify a tag
• [x] Base image should not use latest tag
• [x] Support FROM scratch without a tag
• [x] Support the FROM <image>@<digest> syntax
• [ ] Allow config to specify "allowed" base layers

• [x] Should be followed by exactly 1 parameter (@ sign)

• [x] sudo is not included in the command
• [x] apt-get [install | upgrade | remove] should include a -y flag
• [x] apt-get install commands should include a --no-install-recommends flag
• [x] apt-get install commands should be paired with a rm -rf /var/lib/apt/lists/* in the same layer
• [x] Avoid running apt-get upgrade or apt-get dist-upgrade
• [x] Never run apt-get update without apt-get install on the same line
• [x] apk add commands should include a --no-cache flag or be paired with an --update flag with rm -rf /var/cache/apk/* in the same layer
• [x] apk add support for --virtual flag
• [ ] handle best practices for yum operations and cleanup

• [x] Only a single CMD layer is allowed
• [ ] Better handling of escaped quotes
• [ ] Detect exec format with expected variable substitution

• [x] Format should be key=value

• [x] Only the container port should be listed
• [ ] All ports should be exposed in a single cache layer (line)
• [ ] The same port number should not be exposed multiple times
• [x] Exposed ports should be numeric and in the accepted range

• [x] Format of ENV
• [ ] Best practice of only using a single ENV line to reduce cache layer count

• [x] Command should have at least 2 parameters
• [x] Source command(s) cannot be absolute or relative paths that exist outside of the current build context
• [x] Commands with wildcards or multiple sources require that destination is a directory, not a file
• [ ] If an ADD command could be a COPY, then COPY is preferred
• [ ] Using ADD to fetch remote files is discouraged because they cannot be removed from the layer

• [ ] Implement checking (similar to ADD)
• [ ] Do not COPY multiple files on a single command to best use cache

• [ ] Support

• [ ] Format
• [ ] Any build steps after VOLUME is declare should not change VOLUME contents
• [ ] If JSON format, double quotes are required

• [x] Should be followed by exactly 1 parameter

• [x] Validate that it has exactly 1 parameter
• [x] WORKDIR can only expand variables previously set in ENV commands

• [ ] Support
• [ ] Prevent redefining the built in ARGs (proxy)

• [ ] Support

• [ ] Validate input
• [ ] Only present one time

• [x] Only valid Dockerfile commands are present
• [x] All commands should have at least 1 parameter
• [x] Check that commands are written as upper case commands