-
-
Notifications
You must be signed in to change notification settings - Fork 234
Remove need for '-' flag to force stdin #74
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
Conversation
It seems to work well but now it includes the wordlist as well as the stdin list. I assume this is not the desired result? |
I can replicate it, this is the relevant code block:
Will look to resolve in the next couple of days unless you have a chance to patch! |
Ignore the above - once I posted that I realised it was just in the reverse. Could you please re-test @timkent? The following would ensure good coverage:
I'll look at making an automated test case for this area once we've merged this branch. |
Looks good, I have tested those scenarios. |
Minor filename fix
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All looks good
This patch removes the need for the '-' flag when using stdin and instead allows you to pipe into VHostScan directly at anytime. STDIN is checked prior to a commit.
Tested already on a Windows VM but still requires testing in Kali.