Merge pull request #69 from vivki/main

add js_periodic to parse

Author: vivki

analyze.go

@@ -2186,6 +2186,10 @@ var directives = map[string][]uint{
 		ngxHTTPMainConf | ngxHTTPSrvConf | ngxHTTPLocConf | ngxConfTake1,
 		ngxStreamMainConf | ngxStreamSrvConf | ngxConfTake1,
 	},
+	"js_periodic": {
+		ngxHTTPLocConf | ngxConfAny,
+		ngxStreamSrvConf | ngxConfAny,
+	},
 	"js_preload_object": {
 		ngxHTTPMainConf | ngxHTTPSrvConf | ngxHTTPLocConf | ngxConfTake13,
 		ngxStreamMainConf | ngxStreamSrvConf | ngxConfTake13,

analyze_test.go

@@ -225,14 +225,40 @@ func TestAnalyze_njs(t *testing.T) {
 			blockCtx{"http", "server", "if"},
 			true,
 		},
+		"js_periodic ok in http": {
+			&Directive{
+				Directive: "js_periodic",
+				Args:      []string{"function"},
+				Line:      5,
+			},
+			blockCtx{"http", "location"},
+			false,
+		},
+		"js_periodic not ok in http": {
+			&Directive{
+				Directive: "js_periodic",
+				Args:      []string{"function"},
+				Line:      5,
+			},
+			blockCtx{"http", "location", "if"},
+			true,
+		},
+		"js_periodic ok in stream": {
+			&Directive{
+				Directive: "js_periodic",
+				Args:      []string{"function"},
+				Line:      5,
+			},
+			blockCtx{"stream", "server"},
+			false,
+		},
 	}
 
 	for name, tc := range testcases {
 		tc := tc
 		t.Run(name, func(t *testing.T) {
 			t.Parallel()
 			err := analyze("nginx.conf", tc.stmt, ";", tc.ctx, &ParseOptions{})
-
 			if !tc.wantErr && err != nil {
 				t.Fatal(err)
 			}