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

use msc_init to check modsecurity #275

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

liudongmiao
Copy link

printf is not part of modsecurity, and may not work with -Werror

printf is not part of modsecurity, and may not work with -Werror
@mikelolasagasti
Copy link

This patch is required by downstream Fedora. It is required even if MODSECURITY_LIB and MODSECURITY_INC are set.

Copy link

sonarcloud bot commented Apr 12, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@airween
Copy link
Member

airween commented Jun 4, 2024

@liudongmiao thanks, this is very useful. (I had solved that with a bit similar way in Debian - but this one is more elegant.)

Could you pick up the current status and rebase your tree? It would be nice to run every PR's against our new workflow.

Thanks!

@liudongmiao
Copy link
Author

@airween Thanks.

PR #275 was made two years ago. In last 2 years, modsecurity and modsecurity-nginx have changed.

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.

3 participants