-
Notifications
You must be signed in to change notification settings - Fork 103
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
sympa_msg.pl crashed with an include_users_ldap_2level #12
Comments
Yoann, could you please show error log entries just before dying? |
@ikedas It looks like the same bug I reported in #11012. |
Hi, thanks for your answer :)
@sivertkh i think it's the same bug too ! |
@ydelattre, could you show syslog messages? The logs you have shown seem stderr outputs under tmp/ spool. |
Here it is :
|
@ydelattre, does this patch stop dying? diff --git a/src/lib/Sympa/List.pm b/src/lib/Sympa/List.pm
index 9b363c7..36ab590 100644
--- a/src/lib/Sympa/List.pm
+++ b/src/lib/Sympa/List.pm
@@ -5617,6 +5617,7 @@ sub _include_users_ldap_2level {
'fliter2', $filter2,
'ldap_attrs2', $ldap_attrs2
};
+ next;
}
## returns a reference to a HASH where the keys are the DNs |
Thanks for your reactivity ! |
@ikedas this patch solve the issue ! Thanks a lot :) |
…el if no object was found on the server.
Thanks for reporting bug. Closed. |
Hi,
We encounter some problem with some lists when the second query from an include_users_ldap_2level try to query an object that doesn't exist in our LDAP.
First : task_manager.pl died when he was trying to sync those lists.
Second : if we send a mail to those lists, sympa_msg.pl dies too.
Third : this makes "review subscriber" page unavailable to the user.
From the sympa log :
I know that our LDAP should be clean and contain no reference to a missing object, but is it possible, for this case, to catch an error and not raise an exception ?
Thanks a lot,
Yoann
The text was updated successfully, but these errors were encountered: