On Friday 01 October 2010 15:22:09 Ralf Haferkamp wrote:
Hi,
find yet another release of the patches attached. It adresses the
remaining issues we discussed in IRC:
- included the Simo's style fixes and rearrangements to better match
the overall sssd code style
- Errors detected in sdap_process_group_members() are not considered
fatal (apart from ENOMEM conditions, where it doesn't seem to make a
lot of sense to continue).
We just log an error message an skip to the next member. This could
be improved a bit by stopping to send new LDAP request when
sdap_get_generic_recv() error code e.g. indicates that the LDAP
connection is permanently broken. What error code would that be,
EIO? But this should probably better addressed together with #633.
Any news on
this? Is the patch now acceptable?
regards,
Ralf