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

Plugins? #10

Closed
loukaniko85 opened this issue Jul 19, 2024 · 8 comments · Fixed by #14
Closed

Plugins? #10

loukaniko85 opened this issue Jul 19, 2024 · 8 comments · Fixed by #14

Comments

@loukaniko85
Copy link

loukaniko85 commented Jul 19, 2024

Hi Team - anyway to get modsecurity plugins to function?

-- Want to jump to this from the modsecurty-crs container, but the plugins are holding me back.

@fzipi
Copy link
Member

fzipi commented Jul 19, 2024

What about mounting the proper directories inside the container?

@fzipi
Copy link
Member

fzipi commented Jul 19, 2024

See https://github.com/coreruleset/coraza-crs-docker/blob/main/src/templates/Caddyfile#L24-L30 for example. You should only need to bind mount the directory where you downloaded your plugins to the /opt/coraza/owasp-crs/plugins/ directory.

@loukaniko85
Copy link
Author

loukaniko85 commented Jul 19, 2024

I've tried this, not sure the plugins get loaded. Same plugins folder as my modsec container, which works.

image

@fzipi
Copy link
Member

fzipi commented Jul 22, 2024

Do you have nextcloud installed? Can you test?

@loukaniko85
Copy link
Author

Of course I do; I am using the modsecurity-crs container at the moment, including the nextcloud plugin, as above, and its working as expected. Ive unmapped and remapped the plugins folder to test, and enable and disabled the plugin in the config itself. Works as expected with this container, certain features of nextcloud are denied with the plugin disabled.

Though with the coraza-crs container, doing the same tests, it never seems to be enabled and the certain features of nextcloud are always denied.

@fzipi
Copy link
Member

fzipi commented Jul 22, 2024

Thanks for confirming! I'll check it out and see what is happening then.

@loukaniko85
Copy link
Author

Happy to. Here if you need anything further.

@fzipi
Copy link
Member

fzipi commented Aug 3, 2024

Did some reorganization and better output and docs in #14.

You can start testing as soon as it is merged and the new container is out!

@fzipi fzipi closed this as completed in #14 Aug 3, 2024
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 a pull request may close this issue.

2 participants