Disable dogtag cert publishing

Dogtag had only one switch, ca.publish.enable, for both CRLs and certs.

Since cert publishing is not used in IPA it should be disabled to
avoid false positives in the logs.

https://pagure.io/freeipa/issue/7522

Signed-off-by: Rob Crittenden <rcritten@redhat.com>
Reviewed-By: Fraser Tweedale <ftweedal@redhat.com>
Reviewed-By: Alexander Bokovoy <abokovoy@redhat.com>
This commit is contained in:
Rob Crittenden
2019-09-24 15:31:38 -04:00
parent 7f5b826e6f
commit f709ca977e
2 changed files with 24 additions and 0 deletions

View File

@@ -922,6 +922,7 @@ class CAInstance(DogtagInstance):
self.config, quotes=False, separator='=') as ds:
# Enable file publishing, disable LDAP
ds.set('ca.publish.enable', 'true')
ds.set('ca.publish.cert.enable', 'false')
ds.set('ca.publish.ldappublish.enable', 'false')
# Create the file publisher, der only, not b64

View File

@@ -459,6 +459,28 @@ def ca_add_default_ocsp_uri(ca):
return True # restart needed
def ca_disable_publish_cert(ca):
logger.info('[Disabling cert publishing]')
if not ca.is_configured():
logger.info('CA is not configured')
return False
value = directivesetter.get_directive(
paths.CA_CS_CFG_PATH,
'ca.publish.cert.enable',
separator='=')
if value:
return False # already set; restart not needed
directivesetter.set_directive(
paths.CA_CS_CFG_PATH,
'ca.publish.cert.enable',
'false',
quotes=False,
separator='=')
return True # restart needed
def upgrade_ca_audit_cert_validity(ca):
"""
Update the Dogtag audit signing certificate.
@@ -2072,6 +2094,7 @@ def upgrade_configuration():
ca_configure_lightweight_ca_acls(ca),
ca_ensure_lightweight_cas_container(ca),
ca_add_default_ocsp_uri(ca),
ca_disable_publish_cert(ca),
])
if ca_restart: