feat: replace old readable string extraction with new filtered multi-string support
Please sign (check) the below before submitting the Pull Request:
[ X ] I have signed the ntop Contributor License Agreement at https://github.com/ntop/legal/blob/main/individual-contributor-licence-agreement.md [ X ] I have read the contributing guidelines at https://github.com/ntop/nDPI/blob/dev/CONTRIBUTING.md [ ] I have updated the documentation (in doc/) to reflect the changes made (if applicable)
Link to the related issue: References https://github.com/ntop/nDPI/issues/2708
Describe changes:
Feat: add extract_readable_strings with advanced filtering; refactor string handling
- Introduces a new function to retrieve multiple human-readable substrings from a payload, with optional filter callback support (e.g., for FQDN detection).
- Replaces the old function with the new approach.
- Increases size from 32 to 128 bytes to support longer string concatenations.
- Renames all occurrences of to the correct spelling .
- Replaces all calls to with , reflecting the new logic.
Quality Gate passed
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
@fabiodepin, could you take a look at the errors in the regression tests, please?
@fabiodepin, could you take a look at the errors in the regression tests, please?
Hi, sure. I'll check and get back to you.