Skip to content

[stable23] Support LDAP dns longer than 255 characters #30238

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
come-nc merged 12 commits into from
Dec 14, 2021
Merged

[stable23] Support LDAP dns longer than 255 characters #30238

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
fe3203a
Support LDAP dns longer than 255 characters
come-nc Nov 2, 2021
514b340
Change column names to ldap_dn and ldap_dn_hash and add migration
come-nc Nov 4, 2021
54007f9
Fix variable names
nickvergessen Nov 4, 2021
189852b
Fixed migration step for user_ldap
come-nc Nov 4, 2021
a0db2a2
Put back length check to have a clear error
come-nc Nov 4, 2021
2336a18
Fixes in migration step
come-nc Nov 4, 2021
98476a9
Add the columns and alter the index in Version1010Date20200630192842
come-nc Nov 18, 2021
fbed00b
Make sure that hash function returns a string
come-nc Nov 18, 2021
9e33d51
Add an index for directory_uuid as well
come-nc Nov 18, 2021
e7a7691
Use clearer names for variables
come-nc Nov 18, 2021
b5b080a
Add missing copyright headers in migration steps
come-nc Dec 9, 2021
6b10702
Bump user_ldap version to make sure the migration runs
come-nc Dec 9, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Put back length check to have a clear error 
Signed-off-by: Côme Chilliet <[email protected]>
  • Loading branch information
@come-nc
come-nc committed Dec 13, 2021
commit a0db2a248c59bb2dabfe5de76cd2cf0ec1028bbe
11 changes: 11 additions & 0 deletions apps/user_ldap/lib/Mapping/AbstractMapping.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -355,6 +355,17 @@ public function getList($offset = null, $limit = null) {
* @return bool
*/
public function map($fdn, $name, $uuid) {
if (mb_strlen($fdn) > 4096) {
\OC::$server->getLogger()->error(
'Cannot map, because the DN exceeds 4096 characters: {dn}',
[
'app' => 'user_ldap',
'dn' => $fdn,
]
);
return false;
}

$row = [
'ldap_dn_hash' => $this->getDNHash($fdn),
'ldap_dn' => $fdn,
Expand Down