Skip to content

Support targeted container assessments over Docker transport - #82

Open
je4yfc wants to merge 6 commits into
dev-sec:masterfrom
je4yfc:run-against-container
Open

Support targeted container assessments over Docker transport#82
je4yfc wants to merge 6 commits into
dev-sec:masterfrom
je4yfc:run-against-container

Conversation

@je4yfc

@je4yfc je4yfc commented Sep 8, 2026

Copy link
Copy Markdown

Adds support for -t docker://<container_id>, assessing the selected container and its image from an InSpec runner container. This PR is stacked on top of PR #81

  • Reuses existing checks with Docker API metadata and commands executed inside the target.
  • Provides explicit skips for unavailable host evidence, tools, or permissions.
  • Requires no Docker CLI or socket mount inside the target container.
  • Fixes process listings to include the PID column required by Docker.
  • Documents the containerized invocation and partial assessment coverage.

Closes #36, enables useful checks against a selected container over Docker transport instead of skipping the entire profile.

AI Disclosure: Mapping from the PDF to tasks, some linting fixes, and documentation for this Ansible role were completed with assistance from Google Gemini. I'm sorry, I type like a moron who never passed high school english. I reviewed it all myself, and tested it.

Update control numbering, add missing recommendations, and remove
obsolete checks. Preserve existing title and description conventions
and the established skips for controls requiring manual review.

Fix Docker group membership and missing healthcheck handling found
during SSH testing. Update profile metadata and documentation.

Signed-off-by: Joseph Erdosy <168128519+je4yfc@users.noreply.github.com>
Restore executable checks for manual recommendations and add package,
image-history, port, and artifact-signature checks. Fix audit parsing,
daemon detection, and runtime false positives; provide actionable skips
where host evidence is unavailable.

Signed-off-by: Joseph Erdosy <168128519+je4yfc@users.noreply.github.com>
Report container IDs and relevant properties instead of full inspect
objects, preserving existing assertions.

Closes dev-sec#76

Signed-off-by: Joseph Erdosy <168128519+je4yfc@users.noreply.github.com>
@je4yfc
je4yfc force-pushed the run-against-container branch from da5e0e7 to 6574d20 Compare September 8, 2026 02:10
Signed-off-by: Joseph Erdosy <168128519+je4yfc@users.noreply.github.com>
Assess the selected container and its image through docker://, with
actionable skips where host evidence or permissions are unavailable.
Document the containerized invocation and fix Docker process listings.

Closes dev-sec#36

Signed-off-by: Joseph Erdosy <168128519+je4yfc@users.noreply.github.com>
Signed-off-by: Joseph Erdosy <168128519+je4yfc@users.noreply.github.com>
@je4yfc
je4yfc force-pushed the run-against-container branch from 6574d20 to 6150db0 Compare September 8, 2026 02:18
@je4yfc

je4yfc commented Sep 8, 2026

Copy link
Copy Markdown
Author

Ah Crap - I forgot your request is that:

Please do not update the CHANGELOG.md. We handle that as part of the release process.

I thought I was being helpful.
Let me know if you want it removed and I'll role that commit back.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Test running docker via -t docker://

1 participant