feat: override ssl_ocsp in code generator config

Author: dareste

analyze.go

@@ -19,20 +19,20 @@ package crossplane
 // Update for OSS, filter in config is the directives not in https://nginx.org/en/docs/dirindex.html but in source code.
 // Override in config is for the "if" directive. We create a bitmask ngxConfExpr for it in crossplane, which is not in source code.
 //go:generate sh -c "sh ./scripts/generate/generate.sh --url https://github.com/nginx/nginx.git --config-path ./scripts/generate/configs/oss_latest_config.json > ./analyze_oss_latest_directives.gen.go"
-//go:generate sh -c "sh ./scripts/generate/generate.sh --url https://github.com/nginx/nginx.git --config-path ./scripts/generate/configs/oss_126_config.json --branch branches/stable-1.26 > ./analyze_oss_126_directives.gen.go"
-//go:generate sh -c "sh ./scripts/generate/generate.sh --url https://github.com/nginx/nginx.git --config-path ./scripts/generate/configs/oss_124_config.json --branch branches/stable-1.24 > ./analyze_oss_124_directives.gen.go"
+//go:generate sh -c "sh ./scripts/generate/generate.sh --url https://github.com/nginx/nginx.git --config-path ./scripts/generate/configs/oss_126_config.json --branch stable-1.26 > ./analyze_oss_126_directives.gen.go"
+//go:generate sh -c "sh ./scripts/generate/generate.sh --url https://github.com/nginx/nginx.git --config-path ./scripts/generate/configs/oss_124_config.json --branch stable-1.24 > ./analyze_oss_124_directives.gen.go"
 
 // Update for lua, override is for the lua block directives, see https://github.com/nginxinc/nginx-go-crossplane/pull/86.
 //go:generate sh -c "sh ./scripts/generate/generate.sh --url https://github.com/openresty/lua-nginx-module.git --config-path ./scripts/generate/configs/lua_config.json  --path ./src > ./analyze_lua_directives.gen.go"
 
-// Update for otel. Filter is for some directives withou context.
+// Update for otel. Filter is for some directives without context.
 // Otel provides its own config handler for some directives and they don't have context. Currently we don't support them.
 //go:generate sh -c "sh ./scripts/generate/generate.sh --url https://github.com/nginxinc/nginx-otel.git --config-path ./scripts/generate/configs/otel_config.json --branch main > ./analyze_otel_directives.gen.go"
 
 // Update for NAP v4 and v5.
 // NAP is a private module. Please ensure you have correct access and put the url.
 // and branch of it in environment variable NAP_URL, NAP_V4_BRANCH, and NAP_V5_BRANCH.
-// Override is for flag dirctives. NAP used ngxConfTake1 for flag directives, we change them to ngxConfFlag in crossplane.
+// Override is for flag directives. NAP used ngxConfTake1 for flag directives, we change them to ngxConfFlag in crossplane.
 // NAP v4
 //go:generate sh -c "sh ./scripts/generate/generate.sh --url $NAP_URL --config-path ./scripts/generate/configs/nap_v4_config.json --branch $NAP_V4_BRANCH --path ./src > analyze_appProtectWAFv4_directives.gen.go"
 // NAP v5

scripts/generate/configs/nplus_R33_config.json

@@ -119,7 +119,11 @@
         "enforce_initial_report": [["ngxMgmtMainConf","ngxConfFlag"]],
         "license_token": [["ngxMgmtMainConf","ngxConfTake1"]],
         "state_path": [["ngxMgmtMainConf","ngxConfTake1"]],
-        "zone_sync": [["ngxStreamSrvConf","ngxConfNoArgs"]]
+        "zone_sync": [["ngxStreamSrvConf","ngxConfNoArgs"]],
+        "ssl_ocsp": [
+            ["ngxHTTPMainConf", "ngxHTTPSrvConf", "ngxConfTake1"],
+            ["ngxStreamMainConf", "ngxStreamSrvConf", "ngxConfTake1"]
+        ]
     },
 
     "matchFuncComment":"MatchNginxPlusR33 contains directives in Nginx Plus R33 source code(including GEOIP, Perl, and XSLT)"

scripts/generate/configs/nplus_R34_config.json

@@ -137,8 +137,11 @@
         "redirect_uri": [["ngxHTTPOIDCConf", "ngxConfTake1"]],
         "scope": [["ngxHTTPOIDCConf", "ngxConfTake1"]],
         "session_store": [["ngxHTTPOIDCConf", "ngxConfTake1"]],
-        "session_timeout": [["ngxHTTPOIDCConf", "ngxConfTake1"]]
-
+        "session_timeout": [["ngxHTTPOIDCConf", "ngxConfTake1"]],
+        "ssl_ocsp": [
+            ["ngxHTTPMainConf", "ngxHTTPSrvConf", "ngxConfTake1"],
+            ["ngxStreamMainConf", "ngxStreamSrvConf", "ngxConfTake1"]
+        ]
     },
 
     "matchFuncComment":"MatchNginxPlusR34 contains directives in Nginx Plus R34 source code(including GEOIP, Perl, and XSLT)"

scripts/generate/configs/nplus_R35_config.json

@@ -142,9 +142,12 @@
         "logout_token_hint": [["ngxHTTPOIDCConf", "ngxConfFlag"]],
         "logout_uri": [["ngxHTTPOIDCConf", "ngxConfTake1"]],
         "post_logout_uri": [["ngxHTTPOIDCConf", "ngxConfTake1"]],
-        "userinfo": [["ngxHTTPOIDCConf", "ngxConfTake1"]]
-
-    },
+        "userinfo": [["ngxHTTPOIDCConf", "ngxConfTake1"]],
+        "ssl_ocsp": [
+            ["ngxHTTPMainConf", "ngxHTTPSrvConf", "ngxConfTake1"],
+            ["ngxStreamMainConf", "ngxStreamSrvConf", "ngxConfTake1"]
+        ]
+},
 
     "matchFuncComment":"MatchNginxPlusR35 contains directives in Nginx Plus R35 source code(including GEOIP, Perl, and XSLT)"
 }

scripts/generate/configs/oss_124_config.json

@@ -38,7 +38,8 @@
         "gzip_no_buffer"
     ],
     "override":{
-        "if":[[ "ngxHTTPSrvConf", "ngxHTTPLocConf", "ngxConfBlock", "ngxConfExpr", "ngxConf1More"]]
+        "if":[[ "ngxHTTPSrvConf", "ngxHTTPLocConf", "ngxConfBlock", "ngxConfExpr", "ngxConf1More"]],
+        "ssl_ocsp": [["ngxHTTPMainConf", "ngxHTTPSrvConf", "ngxConfTake1"]]
     },
     "matchFuncComment":"MatchOss124 contains directives in OSS 1.2.4 source code(including GEOIP, Perl, and XSLT)"
 }

scripts/generate/configs/oss_126_config.json

@@ -38,7 +38,8 @@
         "gzip_no_buffer"
     ],
     "override":{
-        "if":[[ "ngxHTTPSrvConf", "ngxHTTPLocConf", "ngxConfBlock", "ngxConfExpr", "ngxConf1More"]]
+        "if":[[ "ngxHTTPSrvConf", "ngxHTTPLocConf", "ngxConfBlock", "ngxConfExpr", "ngxConf1More"]],
+        "ssl_ocsp": [["ngxHTTPMainConf", "ngxHTTPSrvConf", "ngxConfTake1"]]
     },
     "matchFuncComment":"MatchOss126 contains directives in OSS 1.2.6 source code(including GEOIP, Perl, and XSLT)"
 }