| Package | Affected versions | Patched versions |
|---|---|---|
| praisonai | < 4.5.113 | 4.5.113 |
| praisonai | ≤ 4.5.112 |
PraisonAI's recipe registry publish endpoint writes uploaded recipe bundles to a filesystem path derived from the bundle's internal manifest.json before it verifies that the manifest name and version match the HTTP route. A malicious publisher can place ../ traversal sequences in the bundle manifest and cause the registry server to create files outside the configured registry root even though the request is ultimately rejected with HTTP 400.
This is an arbitrary file write / path traversal issue on the registry host. It affects deployments that expose the recipe registry publish flow. If the registry is intentionally run without a token, any network client that can reach the service can trigger it. If a token is configured, any user with publish access can still exploit it.
The bug is caused by the order of operations between the HTTP handler and the registry storage layer.
RegistryServer._handle_publish() in src/praisonai/praisonai/recipe/server.py:370-426 parses POST /v1/recipes/{name}/{version}, writes the uploaded .praison file to a temporary path, and immediately calls:result = self.registry.publish(tmp_path, force=force)
LocalRegistry.publish() in src/praisonai/praisonai/recipe/registry.py:214-287 opens the uploaded tarball, reads manifest.json, and trusts the attacker-controlled name and version fields:name = manifest.get("name")
version = manifest.get("version")
recipe_dir = self.recipes_path / name / version
recipe_dir.mkdir(parents=True, exist_ok=True)
bundle_name = f"{name}-{version}.praison"
dest_path = recipe_dir / bundle_name
shutil.copy2(bundle_path, dest_path)
def _validate_name(name: str) -> bool:
def _validate_version(version: str) -> bool:
but they are not called before the filesystem write.
publish() returns does the route compare the manifest values with the URL values:if result["name"] != name or result["version"] != version:
self.registry.delete(result["name"], result["version"])
return self._error_response(...)
At that point the out-of-root artifact has already been created. The request returns an error, but the write outside the registry root remains on disk.
Running one of these products? Scan your site to see whether it exposes a vulnerable version.
Scan my site freeVerified vulnerable behavior:
/v1/recipes/safe/1.0.0../../outside-dir400/tmp/praisonai-publish-traversal-poc/outside-dir-1.0.0.praisonThis demonstrates that the write occurs before the consistency check and rollback.
Run the single verification script from the checked-out repository:
cd "/Users/r1zzg0d/Documents/CVE hunting/targets/PraisonAI"
python3 tmp/pocs/poc.py
Expected vulnerable output:
[+] Publish response status: 400
{
"ok": false,
"error": "Bundle name/version (../../[email protected]) doesn't match URL ([email protected])",
"code": "error"
}
[+] Leftover artifact exists: True
[+] Artifact under registry root: False
[+] RESULT: VULNERABLE - upload was rejected, but an out-of-root artifact was still created.
Then verify the artifact manually:
ls -l /tmp/praisonai-publish-traversal-poc/outside-dir-1.0.0.praison
find /tmp/praisonai-publish-traversal-poc -maxdepth 2 | sort
What the script does internally:
.praison bundle whose internal manifest.json contains name = ../../outside-dir./v1/recipes/safe/1.0.0.400 mismatch error.outside-dir-1.0.0.praison was still written outside the configured registry directory.This is a path traversal / arbitrary file write vulnerability in the recipe registry publish flow.
Impacted parties:
Security impact:
Validate manifest.json name and version before any path join or filesystem write. Reject path separators, .., absolute paths, and any value that fails the existing _validate_name() / _validate_version() checks.
Resolve the final destination path and enforce that it remains under the configured registry root before calling mkdir() or copy2(). For example, compare the resolved destination against self.recipes_path.resolve().
Move the URL-to-manifest consistency check ahead of self.registry.publish(...), or refactor publish() so it receives already-validated route parameters instead of trusting attacker-controlled manifest values for storage paths.