File paths from docker transforms should be wrapped in quotes

[subdavis]

I don't think it's docker-py's burden to do anything other than ' '.join(container_args)

If our transforms return strings that might have spaces, they also need to be quoted.

Also random note:

girder_worker/girder_worker/docker/transforms/girder.py

Line 174 in 35b1830

the item to a bind mounted volume, and returns the container path of the directory.

I believe this returns a full file path, not a parent directory path.

[subdavis]

HA! I opened #327 like a year ago then forgot about it.

Looks like this is still an issue. Not sure why that issue was closed.

Reproduce with Dockerfile:

FROM debian:buster-slim

COPY entrypoint.sh /
RUN chmod +x /entrypoint.sh

ENTRYPOINT ["/entrypoint.sh"]

entrypoint.sh:

#!/bin/sh -e

ls "$1"

if any part of the path has spaces and you use any of the girder docker transforms as input paths, you get a no such file or directory.