fix: support relative paths in host path volumes#42
fix: support relative paths in host path volumes#42ttys3 wants to merge 1 commit intoMcrich23:mainfrom
Conversation
- Convert relative paths (e.g., ./data/postgres) to normalized absolute paths - Use URL.standardizedFileURL to remove ./ and ../ components - Pass absolute paths to container run -v command instead of relative paths - Fixes issue where container tool incorrectly treats relative paths as named volume names The container tool requires absolute paths for bind mounts. Previously, relative paths like './data/postgres' were passed directly, causing the tool to validate them as named volume names (which must match ^[A-Za-z0-9][A-Za-z0-9_.-]*$), resulting in validation errors.
I may also need this for #49 |
|
I would be happy to merge it. The PR just needs unit testing first. |
I can help. What kind of tests are you looking for? It seems the |
@oxisto We can put whatever desired into a tmp folder. |
The container tool requires absolute paths for bind mounts. Previously, relative paths like './data/postgres' were passed directly, causing the tool to validate them as named volume names (which must match ^[A-Za-z0-9][A-Za-z0-9_.-]*$), resulting in validation errors.
fix #4