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

Added Humble #705

Merged
merged 21 commits into from
Jan 16, 2024
Merged

Added Humble #705

merged 21 commits into from
Jan 16, 2024

Conversation

cyberamt
Copy link
Contributor

@cyberamt cyberamt commented Jan 8, 2024

This pull request introduces a new scanning modul to artemis called humble.

humble is an open source HTTP-Header security scanner (repo)

I have strongly oriented myself to the implementation of dnsreaper. humble.py has the class Humble which is responsible for the call and additionally the function process_json_data which parses the output.

The parser tries to parse missing security headers so that artemis can display them as interesting findings. If necessary, the parser needs a little refinement so that depricated headers and the like are correctly recognized as "interesting".

This pull request was first made for the artemis-extra-module repo.

Please review the changes and let me know if there are any concerns or suggestions for improvement.

@kazet kazet requested a review from bulkowy January 14, 2024 13:23
artemis/modules/humble.py Outdated Show resolved Hide resolved
artemis/modules/humble.py Outdated Show resolved Hide resolved
artemis/modules/humble.py Outdated Show resolved Hide resolved
bulkowy
bulkowy previously approved these changes Jan 16, 2024
@kazet
Copy link
Member

kazet commented Jan 16, 2024

@cyberamt : thank you for your contribution ❤️

@kazet kazet merged commit f68e10b into CERT-Polska:main Jan 16, 2024
4 checks passed
@cyberamt
Copy link
Contributor Author

Hi, really cool that you were able to improve the code and have memorized it. I would be happy if you could include the account as "cyberamt" in the contributers. I will try to write more merge requests with tools if I like.

@kazet
Copy link
Member

kazet commented Jan 19, 2024

Hello,

added you as a contributor, thanks again for the code!

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.

None yet

3 participants