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 doesn't support multiple domains #21361

Open
WnP opened this issue Dec 27, 2024 · 2 comments · May be fixed by #21362
Open

OIDC doesn't support multiple domains #21361

WnP opened this issue Dec 27, 2024 · 2 comments · May be fixed by #21362

Comments

@WnP
Copy link

WnP commented Dec 27, 2024

Expected behavior and actual behavior:

Allow OIDC client to support multiple domains.

Context

Two replicated Harbor instances, such as:

  • a.harbor.domain.tld
  • b.harbor.domain.tld

The live instance running under harbor.domain.tld with failover between the two instances in case one of them is unavailable.

In this context, both OIDC settings of Harbor instances require both domains:

  • a.harbor.domain.tld and harbor.domain.tld
  • b.harbor.domain.tld and harbor.domain.tld

This way the failover would be smooth and both instances would be accessible via both domains with OIDC support.

The issue

Currently this is not possible because the redirect URL is calculated using extEndpoint.

@WnP WnP linked a pull request Dec 27, 2024 that will close this issue
5 tasks
@reasonerjt
Copy link
Contributor

Could you please elaborate on what you mean by In this context, both OIDC settings of Harbor instances require both domains?

For fail-over case, I assume one domain harbor.domain.tld is sufficient?

@WnP
Copy link
Author

WnP commented Jan 2, 2025

If you have a fail over instance that means you also have replication setup between both instances.

So one domain (harbor.domain.tld) is used for the fail over, while the other one ([a|b].harbor.domain.tld) is used for replication and other ops task (terraform, etc.).

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

Successfully merging a pull request may close this issue.

3 participants