sources/ldap: clean-up certs written from db (#7617)
* sources/ldap: clean-up certs written from db * fix test when certificate is none
This commit is contained in:
parent
6b78e6e283
commit
c0b7d32b36
|
@ -1,5 +1,7 @@
|
||||||
"""authentik LDAP Models"""
|
"""authentik LDAP Models"""
|
||||||
from os import chmod
|
from os import chmod
|
||||||
|
from os.path import dirname, exists
|
||||||
|
from shutil import rmtree
|
||||||
from ssl import CERT_REQUIRED
|
from ssl import CERT_REQUIRED
|
||||||
from tempfile import NamedTemporaryFile, mkdtemp
|
from tempfile import NamedTemporaryFile, mkdtemp
|
||||||
from typing import Optional
|
from typing import Optional
|
||||||
|
@ -189,6 +191,11 @@ class LDAPSource(Source):
|
||||||
raise exc
|
raise exc
|
||||||
server_kwargs["get_info"] = NONE
|
server_kwargs["get_info"] = NONE
|
||||||
return self.connection(server, server_kwargs, connection_kwargs)
|
return self.connection(server, server_kwargs, connection_kwargs)
|
||||||
|
finally:
|
||||||
|
if connection.server.tls.certificate_file is not None and exists(
|
||||||
|
connection.server.tls.certificate_file
|
||||||
|
):
|
||||||
|
rmtree(dirname(connection.server.tls.certificate_file))
|
||||||
return RuntimeError("Failed to bind")
|
return RuntimeError("Failed to bind")
|
||||||
|
|
||||||
@property
|
@property
|
||||||
|
|
Reference in New Issue