View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0027849 | mantisbt | ldap | public | 2021-01-07 07:47 | 2021-03-07 18:28 |
| Reporter | dregad | Assigned To | community | ||
| Priority | normal | Severity | minor | Reproducibility | N/A |
| Status | closed | Resolution | fixed | ||
| Target Version | 2.25.0 | Fixed in Version | 2.25.0 | ||
| Summary | 0027849: LDAP server must be specified as an URI | ||||
| Description | Until now, Mantis allowed setting $g_ldap_server to a hostname (i.e. without specifying the ldap[s]:// protocol), relying on an alternate Admin Checks now report this obsolete usage, and ldap_connect_bind() will log and trigger an error. | ||||
| Tags | No tags attached. | ||||
 |
Fixed in PR https://github.com/mantisbt/mantisbt/pull/1727 |
|
MantisBT: master 94462f8c 2021-01-04 07:29 Sean McBride Committer: dregad Details Diff |
Review of LDAP code; added StartTLS support - added StartTLS support for LDAP, based on illmnec's patch (fixes 0015361). - added new ldap_tls_protocol_min option to specify minimun TLS version. - changed default $g_ldap_protocol_version from 0 to 3 (fixes 0027848). - improved Admin Guide and config_defaults_inc.php PHPDoc comments - corrected log output for ldap_connect, which, despite its name, doesn't actually perform a network connection, according to its docs. - added an Admin Check to ensure that ldap_server config option is in URI form (fixes 0027849). Signed-off-by: Damien Regad <dregad@mantisbt.org> PR https://github.com/mantisbt/mantisbt/pull/1727 |
Affected Issues 0015361, 0027848, 0027849 |
|
| mod - admin/check/check_config_inc.php | Diff File | ||
| mod - config_defaults_inc.php | Diff File | ||
| mod - core/constant_inc.php | Diff File | ||
| mod - core/ldap_api.php | Diff File | ||
| mod - docbook/Admin_Guide/en-US/config/auth.xml | Diff File | ||
| mod - lang/strings_english.txt | Diff File | ||
