apache / incubator-streampark

Make stream processing easier! Easy-to-use streaming application development framework and operation platform.
https://streampark.apache.org/
Apache License 2.0
3.91k stars 1.01k forks source link

[Feature] improve ldap user management #2494

Closed zhoulii closed 1 year ago

zhoulii commented 1 year ago

Search before asking

Description

Streampark support Sign in with LDAP or Sign in with password.

image

Suppose there is a LDAP user {name:user_a,password:123}, when this user login for the first time, Streampark will create a user {name:user_a,password:123}(here we ignore the encryption) in user system. This will lead two security problems:

So I propose to forbid LDAP user Sign in with password. Besides, the LDAP user is usually managed by some LDAP client, reset the password of a LDAP user in Streampark is unreasonable, we should hide the reset button for LDAP user in Streampark webui.

Usage Scenario

No response

Related issues

No response

Are you willing to submit a PR?

Code of Conduct

zhoulii commented 1 year ago

Hi @wolfboys , what's your opinion?

wolfboys commented 1 year ago

Hi @wolfboys , what's your opinion?

great, I agree with you