add revoke by certificate id

evertrust · Mar 5, 2024 · f39e029 · f39e029
1 parent 56ad935
commit f39e029
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/plugins/action/horizon_revoke.py b/plugins/action/horizon_revoke.py
@@ -15,7 +15,7 @@ class ActionModule(HorizonAction):
     TRANSFERS_FILES = True
 
     def _args(self):
-        return ["certificate_pem", "revocation_reason", "skip_already_revoked"]
+        return ["certificate_pem", "certificate_id", "revocation_reason", "skip_already_revoked"]
 
     def run(self, tmp=None, task_vars=None):
         result = super(ActionModule, self).run(tmp, task_vars)

diff --git a/plugins/module_utils/horizon.py b/plugins/module_utils/horizon.py
@@ -176,7 +176,7 @@ def renew(self, certificate_pem, certificate_id, password=None):
 
         return self.post(self.REQUEST_SUBMIT_URL, json)
 
-    def revoke(self, certificate_pem, revocation_reason):
+    def revoke(self, certificate_pem, certificate_id, revocation_reason):
         """
         Revoke a certificate
         :type certificate_pem: Union[str,dict]
@@ -187,6 +187,7 @@ def revoke(self, certificate_pem, revocation_reason):
         json = {
             "workflow": "revoke",
             "certificatePem": self.__load_file_or_string(certificate_pem),
+            "certificateId": certificate_id,
             "revocationReason": revocation_reason,
             "template": {
                 "revocationReason": revocation_reason

diff --git a/plugins/modules/horizon_revoke.py b/plugins/modules/horizon_revoke.py
@@ -25,6 +25,10 @@
         description: The path to the PEM encoded certificate to revoke.
         required: false
         type: path
+  certificate_id:
+    description: The ID of the certificate to revoke.
+    required: false
+    type: str
   revocation_reason:
     description: The reason for revoking the certificate.
     required: false