Header X-Forwarded-Prefix not sent when proxying an API

[erikgb]

Summary

I would expect Kong to set the X-Forwarded-Prefix header when proxying an API. This was registered as issue #1150 and fixed with PR #1180. But it seems like X-Forwarded-Prefix part of this fix has been lost (or removed).

I notice the following constant in the code, but it seems unused:

kong/kong/constants.lua

Line 86 in cb1ed39

FORWARDED_PREFIX = "X-Forwarded-Prefix",

Steps To Reproduce

1. Set up an echo route (DB-Less configuration)

services:
  - name: echo
    url: https://postman-echo.com/get
    routes:
      - name: echo
        paths:
          - /echo

2. Perform a HTTP GET request through Kong

Expected behavior

The X-Forwarded-Prefix header is echoed with the value /echo.

Actual behavior

No X-Forwarded-Prefix header in the echo response

Additional Details & Logs

• Kong version 1.5.1

[erikgb]

Adding Request Transformer Plugin configuration for the service(s) is a possible workaround:

plugins:
  - name: request-transformer
    service: echo
    config:
      add:
        headers:
          - x-forwarded-prefix:/echo

But I would love to not have this additional configuration....

[bungle]

Yes, currently we only do:

X-Forwarded-For
X-Forwarded-Port
X-Forwarded-Proto
X-Forwarded-Host
X-Real-IP

We don't currently support:

X-Forwarded-Prefix
X-Forwarded-By
X-Forwarded-Server
X-Forwarded-SSL
X-Proxied-Https

Out of those that we don't have, I think adding X-Forwarded-Prefix makes most sense.

[bungle]

@erikgb, there is now a PR #5620, any feedback is welcomed!

[adil-bouchaara]

@erikgb can you please tell me which configuration you are using for kong ?

[erikgb]

@erikgb can you please tell me which configuration you are using for kong ?

I am not sure what you are asking for, but we are using DB-less configuration, and as a workaround before Kong 2.1 (with this issue resolved) we have been using the configuration as described in #5602 (comment)

[adil-bouchaara]

I found one https://github.com/hbagdi/deck
thanks