Skip to content

swagger: honour X-Forwarded-Proto header - #69

Merged
bajtos merged 1 commit into
masterfrom
feature/honour-x-forwarded-headers
Oct 23, 2014
Merged

swagger: honour X-Forwarded-Proto header#69
bajtos merged 1 commit into
masterfrom
feature/honour-x-forwarded-headers

Conversation

@bajtos

@bajtos bajtos commented Oct 22, 2014

Copy link
Copy Markdown
Member

Improve the algorithm building baseUrl to honour X-Forwarded-Proto header when it is present.

Close #57

See also shelbys/strong-remoting@c049f13

/to @raymondfeng
/cc @shelbys

Improve the algorithm building `baseUrl` to honour `X-Forwarded-Proto`
header when it is present.
@bajtos

bajtos commented Oct 23, 2014

Copy link
Copy Markdown
Member Author

@raymondfeng ping

@raymondfeng

Copy link
Copy Markdown
Member

LGTM

bajtos added a commit that referenced this pull request Oct 23, 2014
…ders

swagger: honour X-Forwarded-Proto header
@bajtos
bajtos merged commit 8c5354c into master Oct 23, 2014
@bajtos bajtos removed the #review label Oct 23, 2014
@bajtos

bajtos commented Oct 23, 2014

Copy link
Copy Markdown
Member Author

I'll wait with the release until #68 is resolved too.

@bajtos
bajtos deleted the feature/honour-x-forwarded-headers branch October 23, 2014 18:16
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

Successfully merging this pull request may close these issues.

Honor X-Forwarded-Proto [GoDaddy]

2 participants