Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add setting to allow returning the Server header from the backend #787

Merged
merged 1 commit into from
May 28, 2017

Conversation

aledbf
Copy link
Member

@aledbf aledbf commented May 28, 2017

fixes #782

curl -v http://192.168.99.100 -k
* Rebuilt URL to: http://192.168.99.100/
*   Trying 192.168.99.100...
* TCP_NODELAY set
> GET / HTTP/1.1
> Host: 192.168.99.100
> User-Agent: curl/7.52.1
> Accept: */*
> 
< HTTP/1.1 200 OK
< Date: Sun, 28 May 2017 21:46:50 GMT
< Content-Type: text/plain
< Transfer-Encoding: chunked
< Connection: keep-alive
**< Server: echoserver** 
< Strict-Transport-Security: max-age=15724800; includeSubDomains;
< 


Hostname: echoheaders-vt1vr

Pod Information:
	node name:	 minikube
	pod name:	 echoheaders-vt1vr
	pod namespace:	 default
	pod IP:  	 172.17.0.8

Server values:
	server_version=nginx: 1.13.0 - lua: 10008

Request Information:
	client_address=172.17.0.4
	method=GET
	real path=/
	query=
	request_version=1.1
	request_uri=http://192.168.99.100:8080/

Request Headers:
	accept=*/*
	connection=close
	host=192.168.99.100
	user-agent=curl/7.52.1
	x-forwarded-for=192.168.99.1
	x-forwarded-host=192.168.99.100
	x-forwarded-port=443
	x-forwarded-proto=https
	x-original-uri=/
	x-real-ip=192.168.99.1192.168.99.1
	x-scheme=https

Request Body:
	-no body in request-

* Curl_http_done: called premature == 0
* Connection #0 to host 192.168.99.100 left intact

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label May 28, 2017
@k8s-reviewable
Copy link

This change is Reviewable

@coveralls
Copy link

Coverage Status

Coverage increased (+0.008%) to 46.567% when pulling 2f20c6b on aledbf:pass-server-header into 3a9e3fd on kubernetes:master.

@aledbf aledbf merged commit 32f2438 into kubernetes:master May 28, 2017
@aledbf aledbf deleted the pass-server-header branch June 13, 2017 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. nginx
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Set "proxy_pass_header Server;"
4 participants