Conversation
|
Hey @hnakamur ! Is there something specific you are fixing? Can you add something in the description? |
|
@fzipi Thank you for your comment! I added an explanation to the description. |
|
Sorry for the delay. What I see in this diff, for example, is that you are still using the file modsec_audit.log. Is that what you wanted, or still need the directory? |
|
Thank you for taking a time to review my diffs. The audit filename:
|
|
@hnakamur As we are not using those directories, maybe it makes sense for you to also add a |
|
I think we don't need to add a The log directory is needed to be created in |
|
I don't think we need the |
|
Currently the modsecurity-crs-docker/nginx/Dockerfile Line 157 in 443b9e3 Is it OK to change these to |
I would like to use this change to run tests for ModSecurity v2/v3 with ftw