diff mbox

[1/2] nginx-naxsi: depends on BR2_PACKAGE_NGINX_HTTP

Message ID 1480948105-26691-1-git-send-email-johan.oudinet@gmail.com
State Accepted
Headers show

Commit Message

Johan Oudinet Dec. 5, 2016, 2:28 p.m. UTC
This package adds XSS and SQLi protection to Nginx HTTP server.
So, it makes sense iff the HTTP part of nginx is enabled.

Signed-off-by: Johan Oudinet <johan.oudinet@gmail.com>
---
 package/nginx-naxsi/Config.in | 1 +
 1 file changed, 1 insertion(+)

Comments

Thomas Petazzoni Dec. 5, 2016, 8:34 p.m. UTC | #1
Hello,

On Mon,  5 Dec 2016 15:28:24 +0100, Johan Oudinet wrote:
> This package adds XSS and SQLi protection to Nginx HTTP server.
> So, it makes sense iff the HTTP part of nginx is enabled.
> 
> Signed-off-by: Johan Oudinet <johan.oudinet@gmail.com>
> ---
>  package/nginx-naxsi/Config.in | 1 +
>  1 file changed, 1 insertion(+)

Thanks, I've applied both patches. However, I had to fix two things:

 - The indentation was wrong: you used spaces while Config.in entries
   are indented with one tab.

 - You forgot to add references to the autobuilder issues being fixed
   by those patches.

Thanks!

Thomas
diff mbox

Patch

diff --git a/package/nginx-naxsi/Config.in b/package/nginx-naxsi/Config.in
index 5ae5497..e9907aa 100644
--- a/package/nginx-naxsi/Config.in
+++ b/package/nginx-naxsi/Config.in
@@ -1,5 +1,6 @@ 
 config BR2_PACKAGE_NGINX_NAXSI
 	bool "nginx-naxsi"
+        depends on BR2_PACKAGE_NGINX_HTTP
 	# uses pcre, so nginx needs to be built with pcre support
 	select BR2_PACKAGE_PCRE
 	help