These changes basically are to remove referrals from the ldapsearch results. The change is cache sqlite DB schema is required to allow objectGUID AD attribute to work as unique attribute to track LDAP objects.
Details
Details
Diff Detail
Diff Detail
- Repository
- rP pykolab
- Branch
- Changes_for_AD
- Lint
Lint Skipped - Unit
No Test Coverage - Build Status
Buildable 23208 Build 9543: arc lint + arc unit
Event Timeline
Comment Actions
- Use python UUID to convert binary objectGUID to string. This way dont need to chage cache db format. It doesn't look like value of uniqueid returned by from cache is used anywhere (needs to double check), so only forward conversion is needed.
Comment Actions
- Use python UUID to convert binary objectGUID to string. It doesn't look like value of uniqueid returned by from cache is used anywhere (needs to double check), so only forward conversion is needed.
- Also do not strip anything from bytestring attributes (objectGUID) because after stripping conversion to UUID string is not possible.
| pykolab/auth/ldap/__init__.py | ||
|---|---|---|
| 1256 | s/_entry_id/_entry_dn/, but... | |
| pykolab/auth/ldap/__init__.py | ||
|---|---|---|
| 1258 | ... but, didn't you change the returned value format? Maybe you meant _entry_dns.append(_result) in the loop above? | |
Comment Actions
- Use python UUID to convert binary objectGUID to string. It doesn't look like value of uniqueid returned by from cache is used anywhere (needs to double check), so only forward conversion is needed.
Good catch. Here is even more pythonic way removing AD referrals from ldap search results.