-
Notifications
You must be signed in to change notification settings - Fork 778
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
Added flag for single or comma separated stdin input values (#737) #759
Added flag for single or comma separated stdin input values (#737) #759
Conversation
@ehsandeep @Ice3man543 Just wanted to check the current status of the PR and ask if there is anything more I could do to help |
runner/runner.go
Outdated
var numHosts int | ||
// check if a string input has been provided | ||
if r.options.InputTargetHost != "" { |
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.
We should use the logic from goflags
that automatically splits the input into a slice. See StringSliceVarP
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.
You could check how it has been done in other projects of ours (e.g. nuclei
or dnsx
)
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.
Please see my comments.
Using the goflags' StringSlice now.
Done! Just updated the PR. Please let me know if everything is ok. |
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.
lgtm
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.
./httpx -u example.com,google.com -silent
https://google.com
https://example.com
There are a couple of things which I don't know if are ok: