Patch #21137
Rescue network level errors with LDAP auth
| Status: | Closed | Start date: | ||
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: |  Jean-Philippe Lang | % Done: | 0% | |
| Category: | LDAP | |||
| Target version: | 3.2.0 |
Description
Right now, only a small number of possible error cases is rescue inside the AuthSourceLdap model are rescued, namely errors happening on the LDAP protocol itself. This results in plain (and not really helpful) 500 errors on login for many common transient network errors, including an unstable connection between the app and the LDAP server, TLS certificate issues, DNS issues, changed firewalls...
The attached patch against current trunk at r14793 provides a list of exceptions that are explicitly rescued to show a nice error message to the user instead of just failing with an HTTP 500. This should result in much more pleasant experience for people using LDAP auth.
This patch was extracted from Planio.
Associated revisions
Rescue network level errors with LDAP auth (#21137).
Patch by Holger Just.
History
#1
Updated by Toshi MARUYAMA over 5 years ago
- Target version set to 3.2.0
#2
Updated by Jean-Philippe Lang over 5 years ago
- Status changed from New to Closed
- Assignee set to Jean-Philippe Lang
Thanks.