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

Fix some issues #7

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Fix some issues #7

wants to merge 5 commits into from

Conversation

bormaa
Copy link

@bormaa bormaa commented Oct 10, 2022

This script had some issues like it has to be run from home directory and removed certspotter because it wasn't working. Amass had some error in command ".$url" which make it always return empty as it should be .$url. Some path for potential_takeovers had more folder name potential_takeovers in path so fixed that part the final.txt of domains wasn't finally sorted so resorted unique it as it had many duplicates

This script had some issues like it has to be run from home directory and removed certspotter because it wasn't working.
Amass had some error in command ".$url" which make it always return empty as it should be .$url.
Some path for potential_takeovers had more folder name potential_takeovers in path so fixed that part
the final.txt of domains wasn't finally sorted so resorted unique it as it had many duplicates
Describe how is this different from the main code from @Gr1mmie
Replace eyewitness with gowitness
@bormaa
Copy link
Author

bormaa commented Oct 10, 2022

replaced eyewitness with gowitness and script can run now from any directory

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

Successfully merging this pull request may close these issues.

2 participants