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

Unclear documentation for configuring multiple redirect URIs in Azure AD authentication #1318

Open
qdrop17 opened this issue Jun 5, 2024 · 4 comments

Comments

@qdrop17
Copy link

qdrop17 commented Jun 5, 2024

Summary

The process of configuring multiple redirect URIs for Azure AD-enabled authentication is not well documented. The relevant documentation can be found at:

While the file provided contains the necessary information, it is presented as a key-value pair rather than a list. As a result, we are unsure how to configure an additional redirect URI for our external exposure.

kubectl get authconfigs.management.cattle.io azuread -o yaml
accessMode: unrestricted
apiVersion: management.cattle.io/v3
applicationId: xxx
applicationSecret: cattle-global-data:azureadconfig-applicationsecret
authEndpoint: https://login.microsoftonline.com/xxx/oauth2/v2.0/authorize
enabled: true
endpoint: https://login.microsoftonline.com/
graphEndpoint: https://graph.microsoft.com
kind: AuthConfig
metadata:
  annotations:
    auth.cattle.io/azuread-endpoint-migrated: "true"
    management.cattle.io/auth-provider-cleanup: unlocked
  creationTimestamp: "2023-11-21T08:27:18Z"
  generation: 4
  labels:
    cattle.io/creator: norman
  name: azuread
  resourceVersion: "7988248"
  uid: xxx
rancherUrl: https://xxx/verify-auth-azure
status:
  conditions:
  - status: "True"
    type: SecretsMigrated
tenantId: xxx
tokenEndpoint: https://login.microsoftonline.com/xxx/oauth2/v2.0/token
type: azureADConfig

It would be great to clarify how this can be done properly.

Related Issues

rancher/rancher#23671

@martyav
Copy link
Contributor

martyav commented Jun 10, 2024

@qdrop17 unfortunately, the mapping here is one to one, and can't accept a list of values. The one thing docs team can do here is clear up the wording to make that more evident.

@qdrop17
Copy link
Author

qdrop17 commented Jun 12, 2024

@qdrop17 unfortunately, the mapping here is one to one, and can't accept a list of values. The one thing docs team can do here is clear up the wording to make that more evident.

Okay, got it. Do you mind mentioning someone from the development team to clarify this feature? To us, it's unclear if Rancher supports multiple redirect URIs or not. We would greatly appreciate this capability.

@martyav
Copy link
Contributor

martyav commented Jun 12, 2024

@JonCrowther or @samjustus could you address?

To us, it's unclear if Rancher supports multiple redirect URIs or not. We would greatly appreciate this capability.

@JonCrowther
Copy link
Contributor

That's correct. You can modify the 1 listed domain manually, but never have more than one at the same time. As far as supporting multiple, it is on our radar, but hasn't been prioritized yet. If you'd like to leave a comment on rancher/rancher#23671, that would help bring attention to the feature.

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

4 participants