Merge pull request #87 from dakshinai/allow-js_shared_dict_zone

Allow directive js_shared_dict_zone

Author: dakshinai

analyze.go

@@ -2262,6 +2262,10 @@ var directives = map[string][]uint{
 		ngxHTTPMainConf | ngxHTTPSrvConf | ngxHTTPLocConf | ngxConfTake2,
 		ngxStreamMainConf | ngxStreamSrvConf | ngxConfTake2,
 	},
+	"js_shared_dict_zone": {
+		ngxHTTPMainConf | ngxConf1More,
+		ngxStreamMainConf | ngxConf1More,
+	},
 	"js_var": {
 		ngxHTTPMainConf | ngxHTTPSrvConf | ngxHTTPLocConf | ngxConfTake12,
 		ngxStreamMainConf | ngxStreamSrvConf | ngxConfTake12,

analyze_test.go

@@ -261,6 +261,33 @@ func TestAnalyze_njs(t *testing.T) {
 			blockCtx{"stream"},
 			true,
 		},
+		"js_shared_dict_zone in http context ok": {
+			&Directive{
+				Directive: "js_shared_dict_zone",
+				Args:      []string{"zone=foo:1M"},
+				Line:      5,
+			},
+			blockCtx{"http"},
+			false,
+		},
+		"js_shared_dict_zone in stream context ok": {
+			&Directive{
+				Directive: "js_shared_dict_zone",
+				Args:      []string{"zone=foo:1M"},
+				Line:      5,
+			},
+			blockCtx{"stream"},
+			false,
+		},
+		"js_shared_dict_zone not ok": {
+			&Directive{
+				Directive: "js_shared_dict_zone",
+				Args:      []string{"zone=foo:1M"},
+				Line:      5,
+			},
+			blockCtx{"http", "location"},
+			true,
+		},
 	}
 
 	for name, tc := range testcases {