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

Add Alteration Wordlist Config Option #162

Merged
merged 4 commits into from
May 6, 2019

Conversation

fork-while-fork
Copy link
Contributor

Implements #157

amass/core/config.go Outdated Show resolved Hide resolved
amass/core/config.go Outdated Show resolved Hide resolved
@dhauenstein
Copy link
Collaborator

Maybe the example amass_config.ini should be extended with this new parameter option as well?

@fork-while-fork
Copy link
Contributor Author

I think I've addressed all your comments @dhauenstein. The only thing that's missing is being able to specify multiple wordlists in the commandline parameter, but I think we're alright with just exposing that advanced configuration in the config file. Let me know what you think!

@dhauenstein
Copy link
Collaborator

Yeah, that was quick, awesome!
Regarding the multiple wordlists on the commandline: One way is to separate those with a comma, although I'd agree that putting those in the config file might be a cleaner way, especially this won't be an option that most users will ever need.
Cheers!

Copy link
Collaborator

@caffix caffix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the excellent contribution!

@caffix caffix merged commit e28acdb into owasp-amass:develop May 6, 2019
@fork-while-fork fork-while-fork deleted the add_alteration_wordlist branch May 7, 2019 12:38
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.

4 participants