Closed (duplicate)
Project:
Lightweight Directory Access Protocol
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
9 Jan 2012 at 02:26 UTC
Updated:
14 Oct 2014 at 21:07 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
johnbarclay commentedThanks. I can't replicate this, but can see how it is likely caused. I committed the fix to this. Please try head and see if it solves the problem.
Also...
All the direct calls to check_plain() in ldap modules are in ldap_servers/ldap_servers.functions.inc between line 198 and 256. They are mostly applying them to the ldap attributes for the tokenization process. So it could be any attribute. I changed the code to skip attribute values that don't pass the drupal check_plain() function since there is no point in tokenizing them anyway.
If you want to see the attribute that is causing the problem, turn on "Log Detailed LDAP Actions" at admin/config/people/ldap/settings and look in watchdog after trying the test user again. It should produce one record for each attribute value it can't deal with. These are likely images, hashed data, etc.
Comment #2
johnbarclay commentedComment #3
ufku commentedIn my Win2k8 AD these fields cause the issue
objectguid (type: OctetString)
logonhours (type: OctetString)
objectsid (type: Sid)
I think it is due to the data type as these are the only fields that are of type OctetString and Sid.
Original values seem ok(attached) when viewed with active directory explorer utility
Comment #4
vinmassaro commentedWe get the same errors when testing a user at /admin/config/people/ldap/servers/test/server_name. These fields seem to be the problem:
objectguid
objectsid
msexchmailboxsecuritydescriptor
Comment #5
johnbarclay commentedthis is expected because the test is bringing in data from all the fields, some of which are binary and not configured. There needs to be some text saying to expect this error. Patch is welcome. Otherwise just ignore this error on the server testing. If the error comes up outside of testing,
Comment #6
johnbarclay commentedComment #7
shawn dearmond commentedDuplicate of #1971060: LDAP Servers: Warning: htmlspecialchars(): Invalid multibyte sequence in argument in check_plain() - ldap servers