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

DM-38747: Support Kerberos GSSAPI binds to LDAP #734

Merged
merged 1 commit into from
Apr 18, 2023
Merged

Conversation

rra
Copy link
Member

@rra rra commented Apr 18, 2023

Install the required libraries and the k5start program in the container and, if a keytab and krb5.conf file are present in the container, run the application under k5start. (This will be handled by the Helm chart for invocations other than the default uvicorn one.) If ldap.useKerberos is set in the configuration and there is no userDn set, use GSSAPI as the authentication method.

Add documentation for the secret and Helm configuration. Reorganize the LDAP configuration documentation a bit to use subheadings.

Install the required libraries and the k5start program in the
container and, if a keytab and krb5.conf file are present in the
container, run the application under k5start. (This will be handled
by the Helm chart for invocations other than the default uvicorn
one.) If ldap.useKerberos is set in the configuration and there is
no userDn set, use GSSAPI as the authentication method.

Add documentation for the secret and Helm configuration. Reorganize
the LDAP configuration documentation a bit to use subheadings.
Copy link
Member

@athornton athornton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I mean, I have no idea what the right k5start flags are but I'm assuming you know what you're doing there. Very cool to have this capability.

@rra rra merged commit 4979898 into main Apr 18, 2023
@rra rra deleted the tickets/DM-38747 branch April 18, 2023 19:46
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.

2 participants