allow unused wordlist in config file #652
Conversation
|
Thanks for the PR. Any ideas why this PR is being ignored? It would significantly improve my workflow when using ffuf. |
|
Is the maintainer consciously ignoring this Pull Request? |
|
Hi everyone, @n3rada, "consciously ignoring" is not the most appropriate for Open Source tools :), Things in Open Source are done when possible and this will be reviewed as soon as possible. @MaxIsMyName an option for you is building the code locally using Golang |
|
Hey!
Yeah, I have been busy with work, family, other hobbies and bunch of stuff
related and unrelated to infosec.
I’m authoring and maintaining quite a few different projects, both open-
and closed source and there’s a finite time at my disposal.
This individual PR is currently on hold as I’m working on a new feature
that integrates ffuf/pencode to the ffuf, and requires a more generic
approach to making keywords / wordlists flexible.
…On Mon 4. Sep 2023 at 20.59, bsysop ***@***.***> wrote:
Hi everyone,
@n3rada <https://github.com/n3rada>, "consciously ignoring" is not the
most appropriate for Open Source tools :), Things in Open Source are done
when possible and this will be reviewed as soon as possible.
@MaxIsMyName <https://github.com/MaxIsMyName> an option for you is
building the code locally using Golang go build and running it with the
PR.
—
Reply to this email directly, view it on GitHub
<#652 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABH6DJPTRGLHIOLFBTJA5ZDXYYJJ7ANCNFSM6AAAAAAVE22ATU>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
|
Forgive me for the "conscientiousness". I meant that you may didn't want to implement it that way at all and therefore ignored the request. Personally, I use |
|
I ended up merging the pencode feature, but didn't get to the point where the wordlists and keywords should be managed. That's going to be something for the future. So contrary to my previous message, I'm going to accept and merge this pull request so it makes it to the upcoming release. Thanks for the PR! |
This is an attempt to fix issue #572.
The current fix just print the error message to
stderrinstead of adding it to theMultierrordefined.I am open to better alternatives 😁
I also had to remove unused worldlist in the
Config.InputProviders. Otherwise, unused wordlists are still considered as used (in the reports for instance).