Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Manage X-Forwarded-Port on Gateway #8653

Closed
ppgravitee opened this issue Nov 16, 2022 — with Exalate Issue Sync · 1 comment
Closed

Manage X-Forwarded-Port on Gateway #8653

ppgravitee opened this issue Nov 16, 2022 — with Exalate Issue Sync · 1 comment

Comments

Copy link
Contributor

As platform administrator I want to use a reverse proxy in front of the AccessManagement service and be able to specify the listening port of this proxy. As a consequence the AM Gateway have to manage theX-Forwarded-Port header to build the right URL when a redirection has to be done.

Currently, the URL is build using this class : [https://github.com/gravitee-io/gravitee-access-management/blob/master/gravitee-am-gateway/gravitee-am-gateway-handler/gravitee-am-gateway-handler-common/src/main/java/io/gravitee/am/gateway/handler/common/vertx/utils/UriBuilderRequest.java|https://github.com/gravitee-io/gravitee-access-management/blob/master/gravitee-am-gateway/gravitee-am-gateway-handler/gravitee-am-gateway-handler-common/src/main/java/io/gravitee/am/gateway/handler/common/vertx/utils/UriBuilderRequest.java]

h1. How to test:

Use the docker-compose.yml available into the AM project directory docker/compose. Define a custom port for nginx. Try to initiate a login flow, the GW should redirect you on the right login page.

You will need to apply the changes of this PR [https://github.com/gravitee-io/gravitee-access-management/pull/2201|https://github.com/gravitee-io/gravitee-access-management/pull/2201] in 3.15.x to be able to test

@leleueri leleueri added this to the AM - 3.18.13 milestone Dec 5, 2022
@leleueri leleueri self-assigned this Dec 5, 2022
leleueri added a commit to gravitee-io/gravitee-access-management that referenced this issue Dec 5, 2022
leleueri added a commit to gravitee-io/gravitee-access-management that referenced this issue Dec 8, 2022
@exalate-issue-sync exalate-issue-sync bot added the p2 label Dec 8, 2022
@exalate-issue-sync
Copy link

Anastasiia Vasylenko commented: Env: [https://am.management.v3-18-x.gravitee.dev/|https://am.management.v3-18-x.gravitee.dev/|smart-link]
Version: 3.18.13

As discussed with Eric - this PR were tested locally

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants