Ldap urls not managing multiple urls

[plouarn]

Issue submitter TODO list

• I've looked up my issue in FAQ
• I've searched for an already existing issues here
• I've tried running master-labeled docker image and the issue still persists there
• I'm running a supported version of the application which is listed here

Describe the bug (actual behavior)

We can't put a list of ldaps url even if the property name is "urls" with a "s".

in LdapSecurityConfig.java urls is mapped in LdapContextSource ctx like ctx.setUrl(props.getUrls()); that's wrong that must be in ctx.setUrls and props.getUrls() must be a list not a string

Expected behavior

we can't put a list of ldap url in urls property

Your installation details

i run from docker container provectuslabs/kafka-ui:latest kafka-ui-api-v0.7.2.jar

Steps to reproduce

have multiple ldap urls
set all ldap properties. try to put multiple url to urls.

Screenshots

No response

Logs

No response

Additional context

No response

[github-actions]

Hello there plouarn! 👋

Thank you and congratulations 🎉 for opening your very first issue in this project! 💖

In case you want to claim this issue, please comment down below! We will try to get back to you as soon as we can. 👀

[Haarolean]

Hi, this repo is no longer maintained (#4255). Happy to address your issues here: https://github.com/kafbat/kafka-ui (or feel free to submit a PR right away)