jhrozek commented on a pull request
On Tue, Sep 13, 2016 at 01:33:49AM -0700, mzidek-rh wrote:
Makes sense, but I would prefer to add new "else if" branch
with ADMINLIMIT_EXCEEDED specific debug message.
It would also make sense to change the SDAP_SRCH_FLG_SIZELIMIT_SILENT into something more
generic, like SDAP_SRCH_FLG_SILENT because we want to use it for both of these cases.
I agree with both, please send a separate patch (at least you'll have
npatch++), but I need to apply a fix to downstream now..
See the full comment at https://github.com/SSSD/sssd/pull/22#issuecomment-246952366