From: David Mulder <dmulder@suse.com>
Date: Fri, 6 May 2022 15:41:19 +0000 (-0600)
Subject: gpo: Supress error caused by ldap Cert Auto Enroll config
X-Git-Tag: talloc-2.3.4~147
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=144878ce45cad445f04ff180d647317b67c0a2e6;p=thirdparty%2Fsamba.git

gpo: Supress error caused by ldap Cert Auto Enroll config

When the CA url specified on the SYSVOL is 'LDAP:'
this means that configuration should be fetched
from LDAP. This corrects an error message that
was being reported when the URL appeared improper
but really is not. This does not change the
code behavior (it was still working), but
removes the invalid error.

Signed-off-by: David Mulder <dmulder@suse.com>
Revewed-by: Andreas Schneider <asn@samba.org>
---

diff --git a/python/samba/gp_cert_auto_enroll_ext.py b/python/samba/gp_cert_auto_enroll_ext.py
index e6c40462d33..db85c97fe66 100644
--- a/python/samba/gp_cert_auto_enroll_ext.py
+++ b/python/samba/gp_cert_auto_enroll_ext.py
@@ -116,7 +116,7 @@ def obtain_end_point_information(entries):
             ca['name'] = name
             ca['hostname'] = m.group('server')
             ca['auth'] = m.group('auth')
-        else:
+        elif ca['URL'].lower() != 'ldap:':
             edata = { 'endpoint': ca['URL'] }
             log.error('Failed to parse the endpoint', edata)
     end_point_information = \