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

Print url instead of host #50

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

melardev
Copy link

@melardev melardev commented Mar 3, 2021

Hi, not sure how others feel, but I was tired of trying to remember what was the endpoint I ran param miner against, it was printing:

Resuming url bruteforce at -1 on target.com

I changed it to:

Resuming url bruteforce at -1 on https://target.com/some/path?a=b

It was just one line of code changed, hope it was not enough to break something. As far I can tell, the field is only used for print so it should be ok.

@r3l1c7
Copy link

r3l1c7 commented May 27, 2022

I agree this would be an improvement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants