Total vulnerabilities in the database
A mock API configuration for static file serving following the same approach presented in the documentation page, where the server filename is generated via templating features from user input is vulnerable to Path Traversal and LFI, allowing an attacker to get any file in the mock server filesystem. The issue may be particularly relevant in cloud hosted server instances
In sendFileWithCallback
(code) and sendFile
(code) the filePath
variable is parsed using TemplateParser
let filePath = TemplateParser({
shouldOmitDataHelper: false,
// replace backslashes with forward slashes, but not if followed by a dot (to allow helpers with paths containing properties with dots: e.g. {{queryParam 'path.prop\.with\.dots'}})
content: routeResponse.filePath.replace(/\\(?!\.)/g, '/'),
environment: this.environment,
processedDatabuckets: this.processedDatabuckets,
globalVariables: this.globalVariables,
request: serverRequest,
envVarsPrefix: this.options.envVarsPrefix
});
The path extracted from the request parameters used when composing the final file path is not sanitized and is vulnerable to path traversal exploits (e.g. ../../../../../etc/passwd
)
The issue has been tested with mockoon-cli
, using the Docker image mockoon/cli:latest
# Folder setup
mkdir mockoon-test
cd mockoon-test
# put config.json in mockooon-test dir
mkdir static
# Run container
docker run -d --mount type=bind,source=./config.json,target=/data,readonly -v ./static:/static -p 3000:3000 mockoon/cli:latest -d data -p 3000
Browsing directly to http://localhost:3000/static/%2F..%2F..%2F..%2F..%2F..%2Fetc%2Fpasswd
is going to display the /etc/passwd
file in the container filesystem
Software | From | Fixed in |
---|---|---|
![]() |
- | 9.2.0 |
![]() |
- | 9.2.0 |