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

Health check: include LDAP connection active check if setup #34

Open
Tracked by #20
dk1844 opened this issue May 2, 2023 · 0 comments
Open
Tracked by #20

Health check: include LDAP connection active check if setup #34

dk1844 opened this issue May 2, 2023 · 0 comments

Comments

@dk1844
Copy link
Collaborator

dk1844 commented May 2, 2023

After #30 is merged, we are facing a health check issue for LDAP.

By default, if Spring LDAP implementation is present, healthcheck attempts to include it into the checking (fails is LDAP connection fails). Currently, this default behavior is disabled in config in by management.health.ldap.enabled=false.

When enabled, just as-is, the user is presented with:

org.springframework.ldap.CommunicationException: localhost:389; nested exception is javax.naming.CommunicationException: localhost:389 [Root exception is java.net.ConnectException: Connection refused: connect]

Please find a working setup, but consider the ability to set different modes authorization, too (as outlined in #28).

Hints: maybe we are missing config entries in spring.ldap.* (src) or we are missing a dependency org.springframework.ldap:spring-ldap-core (src). Or maybe both.

@dk1844 dk1844 changed the title LDAP connection active if setup (not ready to be implemented) Healthceck: include LDAP connection active check if setup May 2, 2023
@dk1844 dk1844 changed the title Healthceck: include LDAP connection active check if setup Health check: include LDAP connection active check if setup May 2, 2023
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

1 participant