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

Refactor the CheckMK template. #10494

Merged
merged 2 commits into from
Aug 8, 2024
Merged

Conversation

righettod
Copy link
Contributor

Template / PR Information

Hi,

This PR propose a little refactoring of the template to make it more generic to detect the presence of an instance of the CheckMK software, whatever the final path in place.

It is the reason why I added support for up to 5 redirections because, in my test, I seen such case.

Template Validation

I've validated this template locally?

  • YES
  • NO

Test against the following hosts found via shodan:

http://37.97.160.213
https://51.15.21.13
http://52.212.130.205
http://192.34.60.9
https://173.249.53.90
http://163.178.105.86
https://35.197.144.191

image

Additional Details (leave it blank if not applicable)

Shodan query used: https://www.shodan.io/search?query=http.title%3A%22Check_MK%22

image

Additional References:

None

@DhiyaneshGeek DhiyaneshGeek self-assigned this Aug 8, 2024
@DhiyaneshGeek DhiyaneshGeek added the Done Ready to merge label Aug 8, 2024
@pussycat0x
Copy link
Contributor

pussycat0x commented Aug 8, 2024

Hi @righettod Thank you for updating this template with the community and for your contribution to this project. Your efforts are greatly appreciated. Cheers!

@pussycat0x pussycat0x merged commit cbbe1bb into projectdiscovery:main Aug 8, 2024
2 checks passed
@righettod righettod deleted the checkmk_upd branch August 8, 2024 07:15
@righettod
Copy link
Contributor Author

You are amazing, your delay to handle a PR is amazing 💯

Huge respect and thank you to the team 👍

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

Successfully merging this pull request may close these issues.

3 participants