Conversation
| } | ||
| if c.Request().Header.Get(echo.HeaderXForwardedHost) != "" { | ||
| c.Request().Host = c.Request().Header.Get(echo.HeaderXForwardedHost) |
There was a problem hiding this comment.
Getting host from proxied request is little bit more complex and has security nuances. Please see https://.com/labstack/echo/blob/master/ip.go
NB: it is missing tests |
@padremortius Do you want to continue on this PR or shall we close it? |
@lammel, yes. Added some changes and unit-test |
unit tests are only for scheme parsing. this is only small part of that middleware. Currently it introduces new header "forwarded" RFC7239 which I think - if implemented should be integrated into about RFC7239 and Go standard library golang/go#30963 maybe instead of that Regex we could have small parser that takes in header value, goes over that rune/char by rune/char and outputs parsed parts or an error. RFC7239 should not be that complex as spec is not that complex. |
@aldas i'm added unit-test for method ProxyHeaders(). |
Add middleware for support proxy headers (based on code gorilla mux handler https://.com/gorilla/handlers/blob/master/proxy_headers.go)