v3/master
Fix macOS GitHub Actions build: add PCRE2_CFLAGS/LDADD/LDFLAGS
fix: cppcheck warnings with version 2.21.0
Make 's' const reference to avoid the copy
Avoid this unnecessary copy by using a "const" reference. - Reported by Sonarcloud
Fix review comments for inspect_file, main signature, and timeout fallback
Merge commit from fork
Very first commit: libmodsecurity
Clean up CI and libinjection integration changes
Updated Transformation::evaluate signature to allow for in-place updates, removing unnecessary heap allocated copies. - Renamed Transformation::evaluate to Transformation::transform to avoid confusion with Action's overload methods. - Updated Transformation::transform signature to receive the value by reference and perform the transformation inline, if possible. - Some transformations still need to use a temporary std::string to perform their work, and then copy the result back. - Made Transformation::transform methods const and updated Transaction parameter to be const. - Transaction parameter could not be removed because it's used by just a single transformation, UrlDecodeUni. - Removed std::string Action::evaluate(const std::string &exp, Transaction *transaction); which was only implemented by Transformation but was not used from the base class, but only after downcasting to Transformation, so it can just be declared there (and not pollute other actions with a default member implementation -that does nothing- which is never called).
Move seclang-parser.hh supression to the right place
Configure test fixture using CTest for Windows build - Added new test/test_suite.in with list of regression and unit tests previously in Makefile.am, to be shared between Unix and Windows builds. - Updated regression.cc & unit.cc to return the number of failed tests to indicate to CTest that the test failed. Similarly, a crash or unhandled exception terminates the process with a non-zero exit code. - This change doesn't affect running the tests with autotest in Unix builds because this processes test output from custom-test-driver & test-suite.sh, and ignores the exit code of the test runner. - Removed comment in test/test-cases/regression-offset-variable.json as this is not supported by JSON and prevents strict parsers to read and process the file. - Minor change in regression.cc's clearAuditLog to replace std::ifstream with std::ofstream as the mode to open the flag applies to an output stream. - Minor change in unit.cc to simplify code that deletes tests. - Minor changes to test/custom-test-driver to correct usage information.
Add regression rules for test
Fix Valgrind regression test script
Harden Base64 error handling and add hash/base64 unit coverage
Adds scripts to call the tests using valgrind
Adds a regression test strdup to valgrind suppressions list