Skip to content

docs: note Git Bash path conversion silently breaks the /zap/wrk mount - #3924

Open
SalmanDEmad wants to merge 1 commit into
zaproxy:mainfrom
SalmanDEmad:docs-gitbash-mount-note
Open

docs: note Git Bash path conversion silently breaks the /zap/wrk mount#3924
SalmanDEmad wants to merge 1 commit into
zaproxy:mainfrom
SalmanDEmad:docs-gitbash-mount-note

Conversation

@SalmanDEmad

Copy link
Copy Markdown

When running the packaged scans (e.g. zap-full-scan.py) with docker run -v <host>:/zap/wrk/:rw ... from Git Bash / MSYS on Windows, the mount silently fails and the scan reports:

A file based option has been specified but the directory '/zap/wrk' is not mounted

The cause is not ZAP-specific: Git Bash automatically converts POSIX-style paths in command arguments to Windows paths, which mangles the container side of the -v host:container argument to docker run. This is a well-known MSYS behavior that affects any docker run -v invocation.

This PR adds a short > [!NOTE] callout to the "Mounting the Current Directory" section of the docker docs, next to the existing CWD mount examples, listing the three standard workarounds:

  • prefix the container path with an extra slash (//zap/wrk/),
  • set MSYS_NO_PATHCONVERSION=1 for the command, or
  • run the docker run command from PowerShell or CMD instead.

Docs-only change, single file (site/content/docs/docker/about.md).

@psiinon

psiinon commented Jul 25, 2026

Copy link
Copy Markdown
Member

Logo
Checkmarx One – Scan Summary & Detailsefffed7c-0af6-4ec2-9b15-64fd96e046bf

Great job! No new security vulnerabilities introduced in this pull request


Communicate with Checkmarx by submitting a PR comment with @Checkmarx followed by one of the supported commands. Learn about the supported commands here.

@thc202

thc202 commented Jul 25, 2026

Copy link
Copy Markdown
Member

Finding working solutions for all environments is outside the scope of this document.

I'd prefer we didn't have to document all idiosyncrasies of all envs.

@psiinon

psiinon commented Jul 27, 2026

Copy link
Copy Markdown
Member

How about documenting these sort of things on https://www.zaproxy.org/docs/docker/diagnosing-problems/
Of a new FAQ? https://www.zaproxy.org/faq/

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants