NotificationsYou must be signed in to change notification settings
Fork0
Star0

Commit3cf1701

Felipe Zimmerle

committed

test: Adds loading tests also to nginx.

It was trying to match "ModSecurity for Apache.* configured". This patchmakes it expect for "ModSecurity for nginx.* configured" while nginxversion is being used/tested.

1 parent795d6a6 commit3cf1701Copy full SHA for 3cf1701

File tree

1 file changed

-2

lines changed

tests/regression/config
- 00-load-modsec.t

1 file changed

-2

lines changed

`‎tests/regression/config/00-load-modsec.t‎`

Lines changed: 8 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,10 @@`
`2`	`2`	`type=>"config",`
`3`	`3`	`comment=>"module loaded",`
`4`	`4`	`match_log=> {`
`5`		`-error=> [ qr/ModSecurityfor Apache.* configured\./,10 ],`
	`5`	`+error=> {`
	`6`	`+apache=> [ qr/ModSecurityfor Apache.* configured\./,10 ],`
	`7`	`+nginx=> [ qr/ModSecurityfor nginx.* configured\./,10 ],`
	`8`	`+},`
`6`	`9`	`},`
`7`	`10`	`},`
`8`	`11`	`{`
`@@ -18,6 +21,9 @@`
`18`	`21`	`return$conf;`
`19`	`22`	`},`
`20`	`23`	`match_log=> {`
`21`		`-error=> [ qr/ModSecurityfor Apache.* configured\./,10 ],`
	`24`	`+error=> {`
	`25`	`+apache=> [ qr/ModSecurityfor Apache.* configured\./,10 ],`
	`26`	`+nginx=> [ qr/ModSecurityfor nginx.* configured\./,10 ],`
	`27`	`+},`
`22`	`28`	`},`
`23`	`29`	`},`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit3cf1701

File tree

1 file changed

1 file changed

`‎tests/regression/config/00-load-modsec.t‎`

0 commit comments