refactor: remove NGINX support from v2 (#3498)

Author: sanjib2006

Makefile.am

@@ -35,10 +35,6 @@ test: check
 test-regression:
 	(cd tests && $(MAKE) test-regression)
 
-test-regression-nginx:
-	(cd tests && $(MAKE) test-regression-nginx)
-
-
 cppcheck:
 	@cppcheck \
 	    -j `getconf _NPROCESSORS_ONLN 2>/dev/null || sysctl -n hw.ncpu || echo 1` \
@@ -58,8 +54,7 @@ cppcheck:
 	    --inconclusive \
 	    --template="warning: {file},{line},{severity},{id},{message}" \
 	    --error-exitcode=1 \
-		standalone/ \
-	    nginx/
+		standalone/ 
 
 check-static: cppcheck
 

apache2/mod_security2.c

@@ -91,7 +91,7 @@ TreeRoot DSOLOCAL *conn_write_state_whitelist = 0;
 TreeRoot DSOLOCAL *conn_write_state_suspicious_list = 0;
 
 
-#if defined(WIN32) || defined(VERSION_NGINX)
+#if defined(WIN32)
 int (*modsecDropAction)(request_rec *r) = NULL;
 #endif
 static int server_limit, thread_limit;
@@ -235,7 +235,7 @@ int perform_interception(modsec_rec *msr) {
             break;
 
         case ACTION_PROXY :
-#if !(defined(VERSION_IIS)) && !(defined(VERSION_NGINX)) && !(defined(VERSION_STANDALONE))
+#if !(defined(VERSION_IIS)) && !(defined(VERSION_STANDALONE))
             if (msr->phase < 3) {
                 if (ap_find_linked_module("mod_proxy.c") == NULL) {
                     log_level = 1;
@@ -275,7 +275,7 @@ int perform_interception(modsec_rec *msr) {
             /* ENH This does not seem to work on Windows. Is there a
              *     better way to drop a connection anyway?
              */
-            #if !defined(WIN32) && !defined(VERSION_NGINX)
+            #if !defined(WIN32)
             {
                 extern module core_module;
                 apr_socket_t *csd;
@@ -608,10 +608,10 @@ static apr_status_t change_server_signature(server_rec *s) {
     char *server_version = NULL;
 
     /* This is a very particular way to handle the server banner. It is Apache
-     * only. Stanalone and descendants should address that in its specifics
-     * implementations, e.g. Nginx module.
+     * only. Standalone and descendants should address that in its specifics
+     * implementations, e.g. IIS module.
      */
-#if !(defined(VERSION_IIS)) && !(defined(VERSION_NGINX)) && !(defined(VERSION_STANDALONE))
+#if !(defined(VERSION_IIS)) && !(defined(VERSION_STANDALONE))
     if (new_server_signature == NULL) return 0;
 
     server_version = (char *)apache_get_server_version();

apache2/msc_release.h

@@ -52,16 +52,12 @@
 #ifdef	VERSION_IIS
 #define MODSEC_MODULE_NAME "ModSecurity for IIS (STABLE)"
 #else
-#ifdef	VERSION_NGINX
-#define MODSEC_MODULE_NAME "ModSecurity for nginx (STABLE)"
-#else
 #ifdef	VERSION_STANDALONE
 #define MODSEC_MODULE_NAME "ModSecurity Standalone (STABLE)"
 #else
 #define MODSEC_MODULE_NAME "ModSecurity for Apache"
 #endif
 #endif
-#endif
 #define MODSEC_MODULE_VERSION MODSEC_VERSION
 #define MODSEC_MODULE_NAME_FULL MODSEC_MODULE_NAME "/" MODSEC_MODULE_VERSION " (http://www.modsecurity.org/)"
 

apache2/msc_status_engine.c

@@ -361,8 +361,6 @@ int DSOLOCAL msc_beacon_string (char *beacon_string, int beacon_string_max_len)
     modsec = MODSEC_VERSION;
 #ifdef VERSION_IIS
     apache = "IIS";
-#elif VERSION_NGINX
-    apache = "nginx";
 #else
     apache = real_server_signature;
 #endif

configure.ac

@@ -189,7 +189,7 @@ fi
 # Standalone Module
 AC_ARG_ENABLE(standalone-module,
               AS_HELP_STRING([--enable-standalone-module],
-                             [Enable building standalone module.]),
+                             [Enable building standalone module (IIS, test server). Note: NGINX support has been removed.]),
 [
   if test "$enableval" != "no"; then
     build_standalone_module=1
@@ -297,7 +297,6 @@ if test "$build_docs" -eq 1; then
        AC_CONFIG_FILES([doc/doxygen-apache])
     fi
     if test "$build_standalone_module" -eq 1; then
-       AC_CONFIG_FILES([doc/doxygen-nginx])
        AC_CONFIG_FILES([doc/doxygen-iis])
        AC_CONFIG_FILES([doc/doxygen-standalone])
     fi
@@ -937,7 +936,6 @@ AC_CONFIG_FILES([apache2/Makefile])
 fi
 if test "$build_standalone_module" -ne 0; then
 AC_CONFIG_FILES([standalone/Makefile])
-AC_CONFIG_FILES([nginx/modsecurity/config])
 fi
 if test "$build_extentions" -ne 0; then
 AC_CONFIG_FILES([ext/Makefile])

doc/Makefile.am

@@ -6,9 +6,6 @@ iis:
 	$(DOXYGEN) doxygen-iis
 	touch iis.stamp
 
-nginx:
-	$(DOXYGEN) doxygen-nginx
-	touch nginx.stamp
 
 standalone:
 	$(DOXYGEN) doxygen-standalone
@@ -20,8 +17,8 @@ all-local: apache
 endif
 
 if BUILD_STANDALONE_MODULE
-all-local: iis nginx standalone
+all-local: iis standalone
 endif
 
 clean-local:
-	rm -rf apache iis nginx standalone
+	rm -rf apache iis standalone