DevFW-CICD/ingress-nginx-helm
14
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

access_log should be off for internal /configuration endpoint

Browse source
This commit is contained in:
Elvin Efendi 2018-05-31 13:46:59 -04:00
parent c97d6e1110
commit d4e6c0dfd8
2 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
rootfs/etc/nginx/template/nginx.tmpl
View file

@ -541,6 +541,7 @@ http {
{{ if $all.DynamicConfigurationEnabled }} {{ if $all.DynamicConfigurationEnabled }}
location /configuration { location /configuration {
access_log off;
{{ if $cfg.EnableOpentracing }} {{ if $cfg.EnableOpentracing }}
opentracing off; opentracing off;
{{ end }} {{ end }}

4
test/e2e/lua/dynamic_configuration.go
View file

@ -135,7 +135,9 @@ var _ = framework.IngressNginxDescribe("Dynamic Configuration", func() {
Expect(log).ToNot(BeEmpty()) Expect(log).ToNot(BeEmpty())
By("POSTing new backends to Lua endpoint") By("POSTing new backends to Lua endpoint")
Expect(restOfLogs).To(ContainSubstring("a client request body is buffered to a temporary file")) // NOTE(elvinefendi) now that we disabled access log for this endpoint we have to find a different way to assert this
// or maybe delete this test completely and just rely on unit testing of Lua middleware?
//Expect(restOfLogs).To(ContainSubstring("a client request body is buffered to a temporary file"))
Expect(restOfLogs).ToNot(ContainSubstring("dynamic-configuration: unable to read valid request body")) Expect(restOfLogs).ToNot(ContainSubstring("dynamic-configuration: unable to read valid request body"))
}) })