Location scheme should not always correspond to X-Forwarded-Proto

[modernptz]

Since 20.5.0

Expected behavior

It should be possible to make redirect from http to https and vice versa.

Actual behavior

Currently all Location headers which contains absolute uri are being rewritten by HttpResponseFilter with x-forwarded-proto value as uri scheme.

finatra/http/src/main/scala/com/twitter/finatra/http/filters/HttpResponseFilter.scala

Line 94 in b6453a4

request.headerMap.get("x-forwarded-proto") match {

finatra/http/src/main/scala/com/twitter/finatra/http/filters/HttpResponseFilter.scala

Line 114 in b6453a4

if (fullyQualify) s"${RequestUtils.getScheme(request)}:"

[cacoco]

@modernptz the use of x-forwarded-proto is intentional and is only used if the passed value is different than the returned Location header's current scheme is thus intended to do just what you're asking which is be able to switch from http to https (or vice versa) but always based on the x-forwarded-proto value. That is if you're returning an http scheme but the client's originating protocol was actually https.

Are you asking for a way to ignore the x-forwarded-proto header value to be able to return a scheme that is different than the client's originating protocol?