Skip to content

CI improvement: First check syntax & always display error.log #3172

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

Closed
wants to merge 138 commits into from
Closed

CI improvement: First check syntax & always display error.log #3172

wants to merge 138 commits into from

Conversation

marcstern
Copy link

@marcstern marcstern commented Jun 13, 2024

First check syntax, in case no error.log is generated. this displays error on console.
Always show error.log (in sub-menu) after start. Even if error and CI processing stops.

Marc Stern and others added 30 commits July 31, 2023 16:17
Another NULL pointer check
Allow macro expansion in ctl:ruleRemoveTargetByTag
Fix ErrorDocument processing if status is changed during phase 4
status 400 instead of 500 on parsing error
Support for "filename*" in multipart (precedence over "filename").
Fix for DEBUG_CONF compile flag
Marc Stern and others added 18 commits October 4, 2023 16:01
Implemented msre_action_phase_validate()
…The default is to also store length of ARGS names.
Add context info to error message
Allow to store only the length of ARGS values in ARGS_COMBINED_SIZE
Ignore empty action instead of storing it
Log involved collection key in case of DBM write error
Always show error.log (in sub-menu)
@marcstern marcstern changed the title First check syntax, in case no error.log is generated. CI improvement: First check syntax & always display error.log Jun 13, 2024
@marcstern marcstern added the 2.x Related to ModSecurity version 2.x label Jun 13, 2024
@marcstern marcstern requested a review from airween June 13, 2024 08:50
Copy link

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

@marcstern marcstern closed this by deleting the head repository Jul 20, 2024
@marcstern
Copy link
Author

replaced by #3190

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.x Related to ModSecurity version 2.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant