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

Option "use_lookup_dn_username" from ldapauthenticator missing #110

Closed
petimmermann opened this issue Jun 2, 2020 · 2 comments
Closed

Comments

@petimmermann
Copy link

The ldapauthenticator parameter "use_lookup_dn_username" introduced in version 1.3 jupyterhub/ldapauthenticator#105 is set to "true" by default while setting this to "false" is exactly what we need to have the user's sAMAccountName used as jhub username, pvc and pod naming and so on. Unfortunately there's apparently no way to set this option via the helm chart.

We would love if the current version of the helm chart (0.9.0) would also be able to control all the options provided from the new ldapauthenticator version it is provisioning. Is it possible to take the mentioned option into consideration for the jupyterhub helm chart?

Thanks in advance and thanks for the great work on this amazing project in general,
Peter

@consideRatio
Copy link
Member

Hi Peter!

I think you can use the following general configuration mechanisms. We may indeed lack support for configuring it in auth which have been unsustainable to maintain a mapping to other projects from.

See if this can help you!

Either...

# in config.yaml for the helm chart
config:
  c.ClassNameHere.configurable_option_here: value-here

Or...

# in config.yaml for the helm chart
hub:
  extraConfig:
    c.ClassNameHere.configurable_option_here: value-here

@petimmermann
Copy link
Author

Hi @consideRatio,

thanks for the quick reply. Yes, putting this into "extraConfig" works perfectly fine. :)

Thank you!

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

No branches or pull requests

2 participants