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

Dex configuration improvements to suggest/implement #902

Closed
marians opened this issue Mar 11, 2022 · 0 comments
Closed

Dex configuration improvements to suggest/implement #902

marians opened this issue Mar 11, 2022 · 0 comments
Labels
component/dex epic/managed-dex manuel needs/refinement Needs refinement in order to be actionable

Comments

@marians
Copy link
Member

marians commented Mar 11, 2022

There are a bunch of useful configuration settings which we are currently not using with all customers and which might make sense to suggest to them.

Some may have to be part of our initial setup. All may have to be added to internal docs. Some we might want to recommend to customers individually.

Different connectors offer different options, so this is structured by connector.

Microsoft / Azure AD

The microsoft connector is used for customers with Azure AD.

  • onlySecurityGroups: Dex queries the Microsoft API to obtain a list of groups the user is a member of. onlySecurityGroups configuration option restricts the list to include only security groups. By default all groups (security, Office 365, mailing lists) are included.

  • useGroupsAsWhitelist together with a list of groups named explicitly in the groups key, the list of group membership gets limited to this specific set.

  • emailToLowercase: You can use the emailToLowercase (boolean) configuration option to streamline UPNs (user email) from Active Directory before putting them into an ID token. Without this option, it can be tough to match the email claim because a client application doesn’t know whether an email address has been added with capital- or lowercase letters. As we use the email claim value as the username in Kubernetes RBAC, this can help simplify authorization.

  • groupNameFormat: "id" can be used to specify that the groups claim will contain unique group IDs instead of group names. This is only recommended in environments where group names are expected to change, as group IDs are hard to communicate.

OIDC

  • insecureEnableGroups: according to docs, prevents refreshing the group list. This would mean that a group membership cannot be revoked from a user. We should test and verify this.

TODO: look into other connectors used by customers

@marians marians added component/dex epic/managed-dex needs/refinement Needs refinement in order to be actionable labels Mar 11, 2022
@gawertm gawertm closed this as not planned Won't fix, can't repro, duplicate, stale Jun 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/dex epic/managed-dex manuel needs/refinement Needs refinement in order to be actionable
Projects
None yet
Development

No branches or pull requests

3 participants