'search_fields' is not an array

[p-try]

Thank you very much for this very useful plugin! I have noticed that the configuration in the sample file does not work the way it is provided. This line is in config.inc.php.dist:

'search_fields' => ['mail', 'sAMAccountName'],

I wondered for some time why the plugin didn't work for me. Through Wireshark I found out that the LDAP search query always only searched the cn attribute instead of the provided fields. It turned out that 'search_fields' is expected to be a string, not an array, in the Roundcube LDAP implementation. Once I changed it to

'search_fields' => 'sAMAccountName',

everything started working. I suggest changing the sample config accordingly.

[andreashaerter]

Hey, thanks for the report! I indeed ran into the same problem in a new environment using other schemas and you saved me a lot of debugging time. Will prepare a fix within this week :) and I have to investigate why I came to the conclusion it should be possible to use an array here.

Maybe I wanted to create some loop and got interrupted...

[andreashaerter]

Seems to be related: roundcube/roundcubemail#9420