Feeling Lucky
Composer Dependency Changed
PR-4.3-dev
[#38388] - [4.x] Fix LDAP debugging functionality
- Fixed in Code Base
- 30 Jan 2023
- Medium
- Build: 4.2-dev
- # 38388
- Diff
- tatankat:fix_ldap_debug
User tests: Successful: Unsuccessful:
Pull Request for Issue mentioned in #37959 (comment)
Summary of Changes
- Set ldap client log level to 7, using the symfony ldap client library.
- Require a newer LDAP symfony library, which includes a fix for "bug symfony/symfony#46325 [Ldap] Fix LDAP connection options (@buffcode)" (see https://github.com/symfony/symfony/releases/tag/v5.4.9)
Testing Instructions
Toggle LDAP "Debug", do a login and check the error log for debug statements.
Actual result BEFORE applying this Pull Request
No effect when enabling LDAP "Debug".
Expected result AFTER applying this Pull Request
LDAP debug statements in the php error log.
Documentation Changes Required
none
Remark
Not included in one of my other LDAP PRs as this one includes a change to composer.json
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | External Library Composer Change Front End Plugins |
| Labels |
Added:
Composer Dependency Changed
?
|
||
| Labels |
Added:
PR-4.3-dev
|
||
laoneo
- comment
- 30 Jan 2023
Now we have here a conflict.
| Labels |
Removed:
?
|
||
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-01-30 13:15:29 |
| Closed_By | ⇒ | laoneo |
laoneo
- comment
- 30 Jan 2023
Thanks!
I moved to a more recent version which fixes another bug in symfony/ldap (see the commit message), so we have more correct logging with #37959
You can test with the openldap docker as described in the testing procedure for 4.3