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 bad nuclei config name #1098

Merged
merged 1 commit into from
Dec 5, 2023
Merged

Fix bad nuclei config name #1098

merged 1 commit into from
Dec 5, 2023

Conversation

psyray
Copy link
Contributor

@psyray psyray commented Dec 4, 2023

Nuclei config parameter was named use_conf in scan engine config, but in the code the parameter checked is use_config, so it couldn't work.

While I'm fixing this bug I've normalize the name according to the other ones for naabu, amass and subfinder
image

@psyray psyray self-assigned this Dec 4, 2023
Copy link
Contributor

@AnonymousWP AnonymousWP left a comment

Choose a reason for hiding this comment

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

Assuming you tested & confirmed it's working: approved.

Copy link
Owner

@yogeshojha yogeshojha left a comment

Choose a reason for hiding this comment

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

good one!

@yogeshojha yogeshojha merged commit e9771c5 into yogeshojha:master Dec 5, 2023
3 of 5 checks passed
@psyray psyray deleted the fix-nuclei-config branch December 11, 2023 17:57
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.

3 participants