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

Cosmetic issue in error message with ldap driver #132

Closed
aepli opened this issue Nov 29, 2017 · 1 comment
Closed

Cosmetic issue in error message with ldap driver #132

aepli opened this issue Nov 29, 2017 · 1 comment
Labels
Milestone

Comments

@aepli
Copy link
Contributor

aepli commented Nov 29, 2017

Hello,

During a test to automatically add users to a list, I made a mistake in the user name and got the following error messages:

Failed to bind to LDAP server ldaps://ldaps://ldapsrv.domain.tld: (49) Invalid credentials

The problem comes from initializing the $host_entry variable in the Sympa::DatabaseDriver::LDAP package on line 125:

my $host_entry = sprintf '%s://%s', $connection->scheme, $connection->uri;

In my case, uri already contains the "ldaps://" scheme, which causes the scheme to be displayed twice in the sympa log.

Regards.

Debian 9.2
Perl 5.24.1
exim4 4.89
mariadb 15.1
apache2 2.4.25
sympa 6.2.22

sympa.log

Nov 29 10:58:35 hostname wwsympa[3447]: err main::#1638 > main::do_sync_include#16718 > Sympa::List::sync_include#6874 > Sympa::List::_load_list_members_from_include#6104 > Sympa::List::_include_users_ldap_2level#5498 > Sympa::Database::connect#154 > (eval)#154 > Sympa::DatabaseDriver::LDAP::_connect#177 Failed to bind to LDAP server ldaps://ldaps://ldapsrv.domain.tld: (49) Invalid credentials
Nov 29 10:58:35 hostname wwsympa[3447]: err main::#1638 > main::do_sync_include#16718 > Sympa::List::sync_include#6874 > Sympa::List::_load_list_members_from_include#6104 > Sympa::List::_include_users_ldap_2level#5498 > Sympa::Database::connect#158 Can't connect to Database Sympa::DatabaseDriver::LDAP <bind_dn=cn=sympa agent,ou=mail,dc=domain,dc=tld;ca_file=/etc/ssl/certs/ca-certificates.crt;ca_verify=required;host=ldaps://ldapsrv.domain.tld;ssl_ciphers=ALL;ssl_version=tlsv1;use_tls=ldaps>:
Nov 29 10:58:35 hostname wwsympa[3447]: err main::#1638 > main::do_sync_include#16718 > Sympa::List::sync_include#6874 > Sympa::List::_load_list_members_from_include#6104 > Sympa::List::_include_users_ldap_2level#5500 Unable to connect to the LDAP server "ldapsrv.domain.tld"
ikedas added a commit to ikedas/sympa that referenced this issue Nov 30, 2017
@ikedas ikedas added the bug label Nov 30, 2017
@ikedas ikedas added this to the 6.2.24 milestone Nov 30, 2017
ikedas added a commit that referenced this issue Dec 6, 2017
Cosmetic issue in error message with ldap driver #132 by ikedas
@ikedas
Copy link
Member

ikedas commented Dec 6, 2017

Fixed. Thanks for improving Sympa!

@ikedas ikedas closed this as completed Dec 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants