Hi Fabrice Ménard ,
Thank you for working on this. The patch itself looks spot on. I have a few other comments for you, but they're easy to address.
[N] Testing (instructions and automated tests)
[-] Third party code
[Y] Sanity check
If you could provide some testing instructions, that would be really helpful to our testing team. We don't accept issues for integration without them, but I believe it should be possible to test this without CAS itself. Any LDAP server which supports paged results will, I think, display this error during login. At the very least, we can log it to the apache error log to confirm the fix.
We try to provide meaningful summaries for our git commits so as to make life easier for those coming to look at things later on. There's some helpful guidance on appropriate git commits at http://docs.moodle.org/dev/Coding_style#Git_commits and you can also view existing commit messages in Moodle using `git log` to see what I mean.
You don't need to include the error message in the commit either as this is noted in the issue itself.
Thank you for taking the time to both report, and fix this issue. If you need any help with it, feel free to ping me and I'll do my best. Alternatively, if you aren't able to address these issues for some other reason, we can help you to find a sponsor to tidy things up and get it integrated.
All the best and Happy New Year,