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

x-forwarded-port makes little sense #310

Open
akrabat opened this issue Jun 10, 2018 · 3 comments
Open

x-forwarded-port makes little sense #310

akrabat opened this issue Jun 10, 2018 · 3 comments

Comments

@akrabat
Copy link
Member

akrabat commented Jun 10, 2018

Given that I have an API gateway set up like this: wsk api create /base /test GET test -n testapi --response-type http, then I can access it like this:

$ curl https://service.eu.apiconnect.ibmcloud.com/gws/apigateway/api/2fb...be1/base/test

Inspecting __ow_headers, I find this:

"x-forwarded-url": "https://service.eu.apiconnect.ibmcloud.com/gws/apigateway/api/2fb...be1/base/test",
"x-forwarded-port": "8080",

The URL is correct, but the port should be 443.

@mhamann
Copy link
Member

mhamann commented Aug 8, 2019

@akrabat in your opinion, should we just remove the x-forwarded-port header?

@akrabat
Copy link
Member Author

akrabat commented Aug 8, 2019

I think that x-forwarded-port should be correct.

@rabbah
Copy link
Member

rabbah commented Oct 2, 2019

443 in this case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants