calculated forwarded prefix using path and original uri

[ryandawsonuk]

for #314

[spencergibb]

I've never heard of x original uri, can you point me to some resources?

[codecov-io]

Codecov Report

Merging #377 into master will increase coverage by 3.07%.
The diff coverage is 81.25%.

@@            Coverage Diff             @@
##           master     #377      +/-   ##
==========================================
+ Coverage   65.36%   68.44%   +3.07%     
==========================================
  Files           4      124     +120     
  Lines         231     3473    +3242     
  Branches       17      237     +220     
==========================================
+ Hits          151     2377    +2226     
- Misses         68      966     +898     
- Partials       12      130     +118

Impacted Files	Coverage Δ
...ateway/filter/headers/XForwardedHeadersFilter.java	84.84% <81.25%> (ø)
...work/cloud/gateway/support/ConfigurationUtils.java	56.25% <0%> (ø)
...cloud/gateway/filter/LoadBalancerClientFilter.java	91.66% <0%> (ø)
...d/gateway/route/CachingRouteDefinitionLocator.java	81.81% <0%> (ø)
.../handler/predicate/QueryRoutePredicateFactory.java	68.42% <0%> (ø)
...framework/cloud/gateway/event/FilterArgsEvent.java	100% <0%> (ø)
...oud/gateway/actuate/GatewayControllerEndpoint.java	42.85% <0%> (ø)
...ilter/factory/StripPrefixGatewayFilterFactory.java	94.44% <0%> (ø)
.../cloud/gateway/support/ServerWebExchangeUtils.java	89.18% <0%> (ø)
...oud/gateway/filter/WebClientHttpRoutingFilter.java	0% <0%> (ø)
... and 111 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2816605...0f3efb1. Read the comment docs.

[ryandawsonuk]

Seems from a bit of searching to be an nginx thing, which makes sense as I'm running behind an Ingress. I guess this means this solution is not generic and I should close the PR.

[ryandawsonuk]

@spencergibb Would RouteToRequestUrlFilter be a better place to check for and add the prefix header (if an appropriate property is set)?

[spencergibb]

I don't think so.

[ryandawsonuk]

Or maybe it can be done in XForwardedHeadersFilter by getting the Route object from exchange.getAttribute(GATEWAY_ROUTE_ATTR)

[spencergibb]

That seems like a better way.