-
-
Notifications
You must be signed in to change notification settings - Fork 315
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Parameter --output doesn't write a file if using docker container #361
Comments
Oh, wow, that's a really neat idea! The only downside I can think of, is that'd be to a detriment to the security aspect that Docker provides... e.g. if the $PWD happens to be ~, it'll just expose everything the user owns to whatever's in the container. Right now it has no ability to "infect" or "hijack" anything, but the |
I could make it create a temporary directory, e.g. |
I'm doing this (
|
Wow, that's some 1337 CLI stuff you have going in there. I'll try to do that thing with the temporary directory, will show what I have in a PR. |
From a user's perspective: do whatever is neccessary to keep it as simple as possible when using it. So you would need to fix this line: |
I'm using
monolith
in docker container and followed your installation instructions. But when I run this:I'm getting no file written at all on my local disk because the docker container has no volume ($PWD) mounted 😞
This can be fixed by this:
The text was updated successfully, but these errors were encountered: