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

Add actual error to the log if connection to LDAP server fails. #1045

Merged
merged 1 commit into from
Dec 7, 2020

Conversation

racke
Copy link
Contributor

@racke racke commented Nov 29, 2020

This is really helpful to figure out what went wrong with the connection to the LDAP server.

@ikedas ikedas added this to the 6.2.60 milestone Nov 30, 2020
@ikedas ikedas added bug ready A PR is waiting to be merged. Close to be solved labels Dec 1, 2020
@ikedas
Copy link
Member

ikedas commented Dec 2, 2020

This may fix #596

@ikedas ikedas removed this from the 6.2.60 milestone Dec 2, 2020
@ikedas ikedas linked an issue Dec 2, 2020 that may be closed by this pull request
@racke
Copy link
Contributor Author

racke commented Dec 2, 2020

I think that fix is a very good candidate for the next beta.

@ikedas
Copy link
Member

ikedas commented Dec 2, 2020

I agree. #596 is assigned the milestone 6.2.60.

@ikedas ikedas merged commit 854c193 into sympa-community:sympa-6.2 Dec 7, 2020
@racke racke deleted the pr/ldap-logerror branch December 15, 2020 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug ready A PR is waiting to be merged. Close to be solved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ldap ssl connexion no error message
2 participants