remove unneeded empty LDAP search attribute values

[blizzz]

fixes#12086

@jlehtoranta@MrManor@PaulLebmann this should fix the issue as you described it. Would be great if you can test and confirm it is working again.

[MrManor]

Works for me! Thanks. (Only one file DL and tested as have have no user_ldap/tests catalog in my install)

[rullzer]

Makes sense.

[violoncelloCH]

should this be backported to 14 stable? #12086 seems to make it impossible to use Nextcloud in combination with these LDAP-Servers. Or is the range of users affected small enough?
sorry, overlooked the label

[MorrisJobke]

should this be backported to 14 stable? #12086 seems to make it impossible to use Nextcloud in combination with these LDAP-Servers. Or is the range of users affected small enough?

Yes - we already added the backport-request label and this will be back ported within the next days. 👍

[blizzz]

backport PRs are opened

[kesselb]

Is 0 a valid value for ldapQuotaAttribute? empty(0) would evaluate to true.

[blizzz]

Is 0 a valid value for ldapQuotaAttribute? empty(0) would evaluate to true.

If I understand it correctly, an attribute's name at least is one UTF-8 character, so "0" seems possible. However, it is beyond practice to use numbers as attribute names, especially 0 → it is not relevant.

[PaulLebmann]

Works for me too! Thanks a lot!!