search

rapid7 / recog

Pattern recognition for hosts, services, and content
Other
671 stars 199 forks source link

Add fingerprinting for html body #383

Closed jvoisin closed 2 years ago

jvoisin commented 3 years ago

For performance reasons, the fingerprint should only be checked against the first 4k of html.

tsellers-r7 commented 3 years ago

@jvoisin Please complete the standard PR template when you open PRs. This will provide additional context. How did you arrive at the 4k number vs 1k, 2k, 10k, etc?

Also, please run recog_standardize per the documentation here: https://github.com/rapid7/recog/blob/master/CONTRIBUTING.md#fingerprints

This will help ensure the tests pass.