mirror of
https://github.com/ClickHouse/ClickHouse.git
synced 2024-09-20 16:50:48 +00:00
code fix for Ldap Injection
This commit is contained in:
parent
b3f59537ca
commit
68bdcdca80
@ -84,27 +84,6 @@ def rbac_roles(*roles, node=None):
|
|||||||
node.query(f"DROP ROLE IF EXISTS {role}")
|
node.query(f"DROP ROLE IF EXISTS {role}")
|
||||||
|
|
||||||
|
|
||||||
# def verify_ldap_user_exists(server, username, password):
|
|
||||||
# """Check that LDAP user is defined on the LDAP server."""
|
|
||||||
# with By("searching LDAP database"):
|
|
||||||
# ldap_node = current().context.cluster.node(server)
|
|
||||||
# r = ldap_node.command(
|
|
||||||
# f"ldapwhoami -H ldap://localhost -D 'cn={user_name},ou=users,dc=company,dc=com' -w {password}"
|
|
||||||
# )
|
|
||||||
# assert r.exitcode == 0, error()
|
|
||||||
|
|
||||||
|
|
||||||
def verify_ldap_user_exists(server, username, password, check=False):
|
|
||||||
"""Check that LDAP user is defined on the LDAP server."""
|
|
||||||
if check:
|
|
||||||
with By("searching LDAP database"):
|
|
||||||
ldap_node = current().context.cluster.node(server)
|
|
||||||
r = ldap_node.command(
|
|
||||||
f"ldapwhoami -H ldap://localhost -D 'cn={user_name},ou=users,dc=company,dc=com' -w {password}"
|
|
||||||
)
|
|
||||||
assert r.exitcode == 0, error()
|
|
||||||
|
|
||||||
|
|
||||||
def create_ldap_external_user_directory_config_content(
|
def create_ldap_external_user_directory_config_content(
|
||||||
server=None, roles=None, **kwargs
|
server=None, roles=None, **kwargs
|
||||||
):
|
):
|
||||||
|
Loading…
Reference in New Issue
Block a user