eduar-hte

Results 45 comments of eduar-hte

I set up a GitHub workflow to build and test libModSecurity v3 & nginx w/ModSecurity-nginx in my fork's development branch. If and when this PR is merged, it'd be useful...

I updated the changes to the config file and the one that introduces strdup so that they apply only when the MSVC C++ compiler is used, because it may be...

I've just updated the branch to reference that ModSecurity PR 3132 has been merged. The documentation and Docker build no longer references the forked repository in order to build the...

> The most critical issue is the `make check-static` result [here](https://github.com/owasp-modsecurity/ModSecurity/actions/runs/8837628481/job/24290716767?pr=3132#step:9:2097). The fix is trivial, I think that would be enough to modify the [test/cppcheck_suppressions.txt](https://github.com/owasp-modsecurity/ModSecurity/blob/v3/master/test/cppcheck_suppressions.txt#L57) file, and change line number,...

> Because this is your first issue, the CI workflow does not start automatically when you update the PR, so you weren't able to see that there is another `check-static`...

> Can I add these suppressions in the code itself? If that works with your setup, I think it'd improve maintainability of the project, as you wouldn't need to update...

> I tried this in commit [cc24a4f](https://github.com/owasp-modsecurity/ModSecurity/commit/cc24a4fa4f2d640b238cb2fcce259740822022f4). Let's see if it works and what you think. I enabled actions in my fork so I could try this. I found out...

I also tried out a workflow to build the Windows version of the library adding the following configuration in a branch in my fork: ```yaml build-windows: runs-on: ${{ matrix.os }}...

> I enabled actions in my fork so I could try this. I found out that the current configuration used to run cppcheck needs the `--inline-suppr` argument for the inline...

> After merging this PR with the main codebase, we need to decide how to proceed. It seems like Nginx supported on [Windows](http://nginx.org/en/docs/windows.html), but I don't know how do the...