Skip to content
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

docker version report is stored in the image #52

Open
ArwynFr opened this issue May 26, 2021 · 2 comments
Open

docker version report is stored in the image #52

ArwynFr opened this issue May 26, 2021 · 2 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@ArwynFr
Copy link

ArwynFr commented May 26, 2021

Describe the bug
When running insider in docker, the report gets generated in /opt/insider. This is a problem because the report is hard to access once the container has stopped. The image is basically mixing binaries and data. The report should written to a separate directory, for instance /var/insider so one may bind that to another volume, or sent to stdout.

To Reproduce
Steps to reproduce the behavior:

  1. docker run --name container -v /path:/target insidersec/insider -tech csharp -target /target
  2. docker comit container image
  3. docker run -ti --entrypoint=sh image
  4. ls -l /opt/insider
total 7032
-rwxr-xr-x    1 root     root       7012352 Jan 26 14:26 insider
-rw-r--r--    1 root     root         12505 May 26 09:03 report.html
-rw-r--r--    1 root     root          7443 May 26 09:03 report.json
-rw-r--r--    1 root     root        160403 May 26 09:03 style.css

Expected behavior
report placed somewhere else or output to stdout

Desktop (please complete the following information):

  • OS: Windows
  • Version docker latest
docker run insidersec/insider -version
Version: dev
Git commit:
Build date:
OS/Arch: linux/amd64
@ArwynFr ArwynFr added bug Something isn't working help wanted Extra attention is needed labels May 26, 2021
@ArwynFr
Copy link
Author

ArwynFr commented May 26, 2021

Another solution would be to allow customization of the report output files/directory names

@cosad3s
Copy link

cosad3s commented Aug 18, 2021

Yes, linked to #56

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants