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

ldap ssl connexion no error message #596

Closed
goudal-emmk opened this issue Apr 12, 2019 · 4 comments · Fixed by #1045
Closed

ldap ssl connexion no error message #596

goudal-emmk opened this issue Apr 12, 2019 · 4 comments · Fixed by #1045
Labels
bug ready A PR is waiting to be merged. Close to be solved
Milestone
6.2.60

Comments

@goudal-emmk
Copy link

goudal-emmk commented Apr 12, 2019 •
edited by ikedas
Loading

Version

Sympa 6.2.41b.2

Installation method

from source

Expected behavior

I had some errors in the ldaps configuration (fbad ssl version, and some problem with certificate chain.
The only error message is that the connexion failed.

Actual behavior

Errors are generated either by Net::LDAP module or IO::Socket::SSL but none of these errors are treated by sympa who just put a generic message error "connexion failed" and not the precise message from the preceding modules.
What I would expect is to have these message repported by sympa

Additional information
@goudal-emmk
Copy link
Author

Question ?
It's just a bug concerning error messages not beeing caught and displayed !

@racke
Copy link
Contributor

racke commented Apr 18, 2019

I concur about this issue being a bug. The actually LDAP error should bubble up to the Sympa logs.

@ikedas ikedas added bug and removed question labels Apr 18, 2019
@ikedas
Copy link
Member

ikedas commented Sep 17, 2019

@goudal-emmk. Could you please show us detailed information? For example logs when error occurred (we at least have to know when does it occur).

@racke
Copy link
Contributor

racke commented Dec 1, 2020

#1045 might be helpful here.

@ikedas ikedas mentioned this issue Dec 2, 2020
Merged
@ikedas ikedas added the ready A PR is waiting to be merged. Close to be solved label Dec 2, 2020
@ikedas ikedas added this to the 6.2.60 milestone Dec 2, 2020
@ikedas ikedas linked a pull request Dec 2, 2020 that will close this issue
Add actual error to the log if connection to LDAP server fails. #1045
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug ready A PR is waiting to be merged. Close to be solved
Projects
None yet
Milestone
6.2.60
Development

Successfully merging a pull request may close this issue.

Add actual error to the log if connection to LDAP server fails. racke/sympa
3 participants
@racke @ikedas @goudal-emmk