sources/ldap: remove deprecated default
Signed-off-by: Jens Langhammer <jens.langhammer@beryju.org>
This commit is contained in:
parent
51365dba74
commit
023b97aa69
|
@ -31,8 +31,7 @@ def ldap_sync_all():
|
||||||
@CELERY_APP.task(
|
@CELERY_APP.task(
|
||||||
bind=True, base=MonitoredTask, soft_time_limit=60 * 60 * 2, task_time_limit=60 * 60 * 2
|
bind=True, base=MonitoredTask, soft_time_limit=60 * 60 * 2, task_time_limit=60 * 60 * 2
|
||||||
)
|
)
|
||||||
# TODO: remove Optional[str] in 2021.10
|
def ldap_sync(self: MonitoredTask, source_pk: str, sync_class: str):
|
||||||
def ldap_sync(self: MonitoredTask, source_pk: str, sync_class: Optional[str] = None):
|
|
||||||
"""Synchronization of an LDAP Source"""
|
"""Synchronization of an LDAP Source"""
|
||||||
self.result_timeout_hours = 2
|
self.result_timeout_hours = 2
|
||||||
try:
|
try:
|
||||||
|
@ -41,8 +40,6 @@ def ldap_sync(self: MonitoredTask, source_pk: str, sync_class: Optional[str] = N
|
||||||
# Because the source couldn't be found, we don't have a UID
|
# Because the source couldn't be found, we don't have a UID
|
||||||
# to set the state with
|
# to set the state with
|
||||||
return
|
return
|
||||||
if not sync_class:
|
|
||||||
return
|
|
||||||
sync = path_to_class(sync_class)
|
sync = path_to_class(sync_class)
|
||||||
self.set_uid(f"{slugify(source.name)}-{sync.__name__}")
|
self.set_uid(f"{slugify(source.name)}-{sync.__name__}")
|
||||||
try:
|
try:
|
||||||
|
|
Reference in a new issue