Skip to content

fix: missing docker while podman is configured#446

Open
a-oren wants to merge 1 commit intoguacsec:mainfrom
a-oren:err-docker
Open

fix: missing docker while podman is configured#446
a-oren wants to merge 1 commit intoguacsec:mainfrom
a-oren:err-docker

Conversation

@a-oren
Copy link
Copy Markdown
Contributor

@a-oren a-oren commented Mar 30, 2026

Description

Previously, dockerPodmanInfo used a short-circuit OR (||) to choose between docker and podman suppliers. If the docker supplier threw an error (e.g. docker binary not found), the exception propagated and podman was never tried- even when podman was installed and configured. Wrap the docker supplier call in a try/catch so that when docker is
unavailable, execution falls through to the podman supplier instead of failing the entire analysis.

Also fix getSyftEnvs to use delimiter from node:path instead of
the non-existent File.pathSeparator.

Related issues (if any):

  • fixes: #issue_number_goes_here

Checklist

  • I have followed this repository's contributing guidelines.
  • I will adhere to the project's code of conduct.

Additional information

Anything else?

@a-oren a-oren requested a review from Strum355 March 30, 2026 14:53
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.

1 participant