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.

Sign up for GitHub

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

Jump to bottom

Monitoring switch to glauth/ldap #414

Open
shipperizer opened this issue Mar 18, 2024 · 1 comment
Open

Monitoring switch to glauth/ldap #414

shipperizer opened this issue Mar 18, 2024 · 1 comment

Comments

@shipperizer
Copy link
Contributor

in light of the issue patched by the v2.3.2 and the fact glauth/ldap is actively miantained by glauth it might be a good time to drop the current monitoring implementation and shift it directly to https://github.com/glauth/ldap/blob/master/server.go#L68 as it was sugegsted in #351 (comment)
@Fusion
Copy link
Collaborator

Fusion commented Mar 20, 2024

Makes perfect sense. Can we however make sure to use dependency injection, so we can easily track any upstream ldap updates over time?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Fusion @shipperizer