[PR #3] [CLOSED] Add ModSecurity #17

New issue

Closed

opened 2026-02-27 14:58:09 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 14:58:09 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/NginxProxyManager/docker-nginx-full/pull/3
Author: @Renaud11232
Created: 11/12/2021
Status: ❌ Closed

Base: master ← Head: add-modsecurity

📝 Commits (4)

f1bb3f4 Compile and install ModSecurity
2e4b4bb Fix scripts permissions
142335b Use git for the modsecurity build
f4060e3 Merge branch 'master' into add-modsecurity

📊 Changes

8 files changed (+128 additions, -5 deletions)

View changed files

📝 .jenkins/Jenkinsfile (+6 -3)
📝 docker/Dockerfile (+42 -0)
➕ scripts/build-libmaxminddb (+24 -0)
➕ scripts/build-modsecurity (+28 -0)
📝 scripts/build-openresty (+8 -2)
📝 scripts/buildx (+3 -0)
➕ scripts/install-libmaxminddb (+16 -0)
📝 scripts/install-openresty (+1 -0)

📄 Description

Hello,

This PR aims at adding ModSecurity (jc21/nginx-proxy-manager#847), with all options enabled, to nginx-proxy-manager.
Support for this could later be added in the web UI but I think I would already be great to have it installed, so it can be configured manually, if desired.

For this to work, additional libraries needed to be installed and compiled (The version of libmaxminddb available in the buster repositories was too old).

This also makes the image bigger (824MB for linux/amd64) since only copying libmodsecurity files adds a 355MB layer.

I succesfully built this image on my local machine for amd64.

Thanks

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/NginxProxyManager/docker-nginx-full/pull/3 **Author:** [@Renaud11232](https://github.com/Renaud11232) **Created:** 11/12/2021 **Status:** ❌ Closed **Base:** `master` ← **Head:** `add-modsecurity` --- ### 📝 Commits (4) - [`f1bb3f4`](https://github.com/NginxProxyManager/docker-nginx-full/commit/f1bb3f4d6c72b521b9f826a13cc19acfd64f0389) Compile and install ModSecurity - [`2e4b4bb`](https://github.com/NginxProxyManager/docker-nginx-full/commit/2e4b4bb34c85adc19afce429a538c9db06838653) Fix scripts permissions - [`142335b`](https://github.com/NginxProxyManager/docker-nginx-full/commit/142335b672beafe8dbc02bbe565473ca82c7d633) Use git for the modsecurity build - [`f4060e3`](https://github.com/NginxProxyManager/docker-nginx-full/commit/f4060e311696449ae2a39f83db7eec457eb82de8) Merge branch 'master' into add-modsecurity ### 📊 Changes **8 files changed** (+128 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `.jenkins/Jenkinsfile` (+6 -3) 📝 `docker/Dockerfile` (+42 -0) ➕ `scripts/build-libmaxminddb` (+24 -0) ➕ `scripts/build-modsecurity` (+28 -0) 📝 `scripts/build-openresty` (+8 -2) 📝 `scripts/buildx` (+3 -0) ➕ `scripts/install-libmaxminddb` (+16 -0) 📝 `scripts/install-openresty` (+1 -0) </details> ### 📄 Description Hello, This PR aims at adding ModSecurity (jc21/nginx-proxy-manager#847), with all options enabled, to nginx-proxy-manager. Support for this could later be added in the web UI but I think I would already be great to have it installed, so it can be configured manually, if desired. For this to work, additional libraries needed to be installed and compiled (The version of libmaxminddb available in the buster repositories was too old). This also makes the image bigger (824MB for linux/amd64) since only copying libmodsecurity files adds a 355MB layer. I succesfully built this image on my local machine for amd64. Thanks --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>