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

Fix x-forwarded-port mapping #91

Merged
merged 1 commit into from
Dec 26, 2016

Conversation

aledbf
Copy link
Member

@aledbf aledbf commented Dec 26, 2016

replaces #86

$ curl  localhost -H 'Host: foo-98.bar.com'
CLIENT VALUES:
client_address=10.2.9.2
command=GET
real path=/
query=nil
request_version=1.1
request_uri=http://foo-98.bar.com:8080/

SERVER VALUES:
server_version=nginx: 1.9.11 - lua: 10001

HEADERS RECEIVED:
accept=*/*
connection=close
host=foo-98.bar.com
user-agent=curl/7.50.2
x-forwarded-for=::ffff:10.2.9.1
x-forwarded-host=foo-98.bar.com
x-forwarded-port=80
x-forwarded-proto=http
x-real-ip=::ffff:10.2.9.1
BODY:
-no body in request-

$ curl  localhost -H 'Host: foo-98.bar.com' -H 'x-forwarded-port: 81'
CLIENT VALUES:
client_address=10.2.9.2
command=GET
real path=/
query=nil
request_version=1.1
request_uri=http://foo-98.bar.com:8080/

SERVER VALUES:
server_version=nginx: 1.9.11 - lua: 10001

HEADERS RECEIVED:
accept=*/*
connection=close
host=foo-98.bar.com
user-agent=curl/7.50.2
x-forwarded-for=::ffff:10.2.9.1
x-forwarded-host=foo-98.bar.com
x-forwarded-port=81
x-forwarded-proto=http
x-real-ip=::ffff:10.2.9.1
BODY:
-no body in request-

$ curl  localhost -H 'Host: foo-98.bar.com' -H 'x-forwarded-port: 442'
CLIENT VALUES:
client_address=10.2.9.2
command=GET
real path=/
query=nil
request_version=1.1
request_uri=http://foo-98.bar.com:8080/

SERVER VALUES:
server_version=nginx: 1.9.11 - lua: 10001

HEADERS RECEIVED:
accept=*/*
connection=close
host=foo-98.bar.com
user-agent=curl/7.50.2
x-forwarded-for=::ffff:10.2.9.1
x-forwarded-host=foo-98.bar.com
x-forwarded-port=443
x-forwarded-proto=http
x-real-ip=::ffff:10.2.9.1
BODY:
-no body in request-

$ curl  localhost -H 'Host: foo-98.bar.com' -H 'x-forwarded-port: 9000'
CLIENT VALUES:
client_address=10.2.9.2
command=GET
real path=/
query=nil
request_version=1.1
request_uri=http://foo-98.bar.com:8080/

SERVER VALUES:
server_version=nginx: 1.9.11 - lua: 10001

HEADERS RECEIVED:
accept=*/*
connection=close
host=foo-98.bar.com
user-agent=curl/7.50.2
x-forwarded-for=::ffff:10.2.9.1
x-forwarded-host=foo-98.bar.com
x-forwarded-port=9000
x-forwarded-proto=http
x-real-ip=::ffff:10.2.9.1
BODY:
-no body in request-

@aledbf aledbf self-assigned this Dec 26, 2016
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Dec 26, 2016
@k8s-reviewable
Copy link

This change is Reviewable

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 3b4358b on aledbf:fix-x-forwarded-port into ** on kubernetes:master**.

@aledbf aledbf merged commit 8e90fc0 into kubernetes:master Dec 26, 2016
@aledbf
Copy link
Member Author

aledbf commented Dec 26, 2016

self merge to fix error en template that avoids the start of the controller

@aledbf aledbf mentioned this pull request Dec 26, 2016
@aledbf aledbf deleted the fix-x-forwarded-port branch January 13, 2017 11:48
haoqing0110 pushed a commit to stolostron/management-ingress that referenced this pull request Mar 5, 2021
@ytianxia6 ytianxia6 mentioned this pull request Sep 9, 2021
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants