eric-maynard commented on code in PR #2611:
URL: https://github.com/apache/polaris/pull/2611#discussion_r2361262814
##########
site/content/in-dev/unreleased/external-idp.md:
##########
@@ -126,7 +126,7 @@ External authentication is configured via Quarkus OIDC and
Polaris-specific OIDC
### OIDC Tenant Configuration
-At least one OIDC tenant must be explicitly enabled. In Polaris, realms and
OIDC tenants are distinct concepts. An OIDC tenant represents a specific
identity provider configuration (e.g., `quarkus.oidc.idp1`). A [realm]({{% ref
"realm" %}}) is a logical partition within Polaris.
+At least one OIDC tenant (Identity Provider) must be explicitly enabled. In
Polaris, realms and OIDC tenants are distinct concepts. An OIDC tenant
represents a specific identity provider configuration (e.g.,
`quarkus.oidc.idp1`). A [realm]({{% ref "realm" %}}) is a logical partition
within Polaris.
Review Comment:
Per the
[docs](https://quarkus.io/guides/security-oidc-expanded-configuration#tenant-properties)
I think @dimas-b is correct, but I was confused when I read this the first
time, too. Maybe we can just link out to that doc so that readers can better
understand what a tenant means in this context.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
