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

OIDC - redirect URL frontend-uri built from req if left out of config #3457

Closed
wants to merge 7 commits into from
Closed

Conversation

jmntn2000
Copy link
Contributor

In a multi-tenant environment or any instance of multiple host names resolving to same IP the frontend-uri can not be hard coded in the config file for OIDC redirect. If this is left out of the config then it is now build from the request.

@romain-grecourt
Copy link
Contributor

/oca-checked

@tomas-langer
Copy link
Member

Hello @jmntn2000, we have a parallel PR in progress (#3456) that needs a similar change. If you would be OK with it, I would take over your work and finish it once that PR is merged. I will keep your commit in to preserve the contribution.

@tomas-langer tomas-langer self-assigned this Oct 5, 2021
@jmntn2000
Copy link
Contributor Author

Hello @jmntn2000, we have a parallel PR in progress (#3456) that needs a similar change. If you would be OK with it, I would take over your work and finish it once that PR is merged. I will keep your commit in to preserve the contribution.

That is fine with me. Thank you.

@tomas-langer
Copy link
Member

I have created an issue for this: #3583

@tomas-langer
Copy link
Member

Part is resolved by #3584
The second part requires more analysis. I will create a follow up issue for it

@tomas-langer
Copy link
Member

Follow up issue #3590, closing this PR

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

Successfully merging this pull request may close these issues.

3 participants