Feeling Lucky
?
[#32400] - Search by ldap_uid when binding as user (#32399)
- Closed
- 12 Jun 2022
- Medium
- Build: staging
- # 32400
- Diff
- oantby:patch-32399
User tests: Successful: Unsuccessful:
Pull Request for Issue #32399.
Summary of Changes
Switch from searching by search_string, which is described as not required for bind-as-user, to building a filter based on the ldap_uid attribute laid out in settings.
Testing Instructions
- Enable LDAP plugin
- Set to bind directly as the user, and fill out required configuration, leaving
search_stringblank. - set
display_errorstoOn - Go to administrative login
- Attempt to log in as a user who is valid in LDAP, but is not authorized to access Joomla!
Actual result BEFORE applying this Pull Request
Along with the authentication failure message, a php error message is displayed indicating an invalid LDAP filter.
Expected result AFTER applying this Pull Request
Only the intended authentication failure message is displayed.
Documentation Changes Required
None, to my knowledge.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End Plugins |
| Labels |
Added:
?
|
||
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-06-12 19:07:58 |
| Closed_By | ⇒ | zero-24 | |
| Labels |
Added:
?
Removed: ? |
||
i will not merge this PR here into 3.10. i see where you are comming from but in oder to keep b/c with the current implementation i will keep it as it is. please sync the setting of the search string with the ldap_uid setting for now that should result into the same behavior.
thanks :-)