xwikisas/application-ldapuserimport

Don't silently ignore ldap errors

Closed this issue · 0 comments

I my last contribution, I introduced a try..catch statement that would hide ldap exceptions and put an error in the log. Turns out, it's not desirable because we want the process to fail with a lot of noise if there's an issue, so the root cause can be fixed.