-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add instructions for Docker container usage #591
Conversation
Docker related documentation was missing.
@sullo Hello, can you have a look at this PR please :) |
I rarely use Docker so I'm not fully qualified. @tautology0 or @mostafahussein would be better suited (I think) if they can. |
This is looks good to me, Can we just make it Also we can replace this
|
@mostafahussein Good remarks :) I've pushed another commit addressing them. |
@NicolasCARPi Great!, I would like to suggest another example in case someone wanted to save the reports on the host.
You can get more details about reports in nikto through this: https://cirt.net/nikto2-docs/reports.html. |
I will accept the PR when you two are happy 💯 This looks good so far, if we can get the last commented change in... that it for now? |
@mostafahussein I have added more examples of usage. I chose to bind the volume to Note 1: nikto is executed as root in the container so the resulting exported file is owned by the root user of the host system. Note 2: it seems the json export is broken, because the json is invalid, but that's for another issue :) EDIT: I see that the whole file is changed, this is because my system automatically fixed the line endings of the file (remove CRLF line terminators), which I think is a good thing. I might also make another PR to remove some tabs that are present in some lines in the code. EDIT2: oh in fact there is a |
Ok so we good here for now? With some additional issues to open:
|
@sullo Yes and I am working on the user issue and some other modifications, one of them related to the reports file permissions that @NicolasCARPi mentioned and I think the PR will be ready by tomorrow. If you are going to open an issue for the user part don't forget to assign it to me, if not then i will send the PR tomorrow after testing the modifications |
Docker related documentation was missing.