- Merge request !27Issue #3303828: Incorrect escaping filter query when logging in β (Merged) created by googletorp
- First commit to issue fork.
-
bluegeek9 β
committed 70c258b0 on 8.x-4.x authored by
googletorp β
Issue #3303828 by googletorp: Incorrect escaping filter query when...
-
bluegeek9 β
committed 70c258b0 on 8.x-4.x authored by
googletorp β
- Status changed to Fixed
11 months ago 8:20pm 8 January 2024 - πΊπΈUnited States bluegeek9
$escape_filter = ldap_escape('Users (external)', '', LDAP_ESCAPE_FILTER);
dpm($escape_filter); // "Users \28external\29"
$dn_filter= ldap_escape('Users (external)', '', LDAP_ESCAPE_DN);
dpm($dn_filter); // "Users (external)"
Automatically closed - issue fixed for 2 weeks with no activity.
- Merge request !89Issue #3303828 by bluegeek9: Incorrect escaping filter query when logging in β (Merged) created by bluegeek9
- π¨π¦Canada drupalthings
Hi guys,
Sorry if this isn't following the correct procedure, but I figure any information is good information right :). I found I needed to apply this fix at line 648 as well in order to get our ldap queries to work. I can't remember the specifics because it was a few months back, but I believe the same logic holds as outlined in bluegeek9's comment 6 that explains ldapEscapeDn() is being used in a filter context, so should be replaced by ldapEscapeFilter(). Hope this helps.