Revize c839facb

Re #8702 - Started implementing required CertService updates.

src/services/certificate_service.py

@@ -333 +333 @@
             if not self.certificate_repository.delete(cert.certificate_id):
                 Logger.error(f"The certificate has not been deleted 'ID = {cert.certificate_id}'.")
 
-
-    def get_certificates_issued_by(self, unique_id):
+    def get_certificates_issued_by(self, issuer_id):
         """
         Returns a list of all children of a certificate identified by an unique_id.
         Raises a DatabaseException should any unexpected behavior occur.
-        :param unique_id: target certificate ID
+        :param issuer_id: target certificate ID
         :return: children of unique_id
         """
 
         Logger.debug("Function launched.")
 
         try:
-            if self.certificate_repository.read(unique_id) is None:
-                Logger.error(f"No such certificate found 'ID = {unique_id}'.")
-                raise CertificateNotFoundException(unique_id)
+            if self.certificate_repository.read(issuer_id) is None:
+                Logger.error(f"No such certificate found 'ID = {issuer_id}'.")
+                raise CertificateNotFoundException(issuer_id)
         except DatabaseException:
-            Logger.error(f"No such certificate found 'ID = {unique_id}'.")
-            raise CertificateNotFoundException(unique_id)
+            Logger.error(f"No such certificate found 'ID = {issuer_id}'.")
+            raise CertificateNotFoundException(issuer_id)
+
+        return self.certificate_repository.get_all_issued_by(issuer_id)
 
-        return self.certificate_repository.get_all_issued_by(unique_id)
+    def get_certificates_issued_by_filter(self, issuer_id, target_types, target_usages, target_cn_substring, page,
+                                          per_page):
+        """
+        Returns a list of all children of a certificate identified by an unique_id.
+        Filters the results according to target types, usages, cn substring and pagination.
+        Raises a DatabaseException should any unexpected behavior occur.
+        :param issuer_id: target certificate ID
+        :param target_types: filter of types
+        :param target_usages: filter of usages
+        :param target_cn_substring: CN substring
+        :param page: target page or None
+        :param per_page: certs per page or None
+        :return: list of certificates (a page if specified)
+        """
+
+        Logger.debug("Function launched.")
+
+        try:
+            if self.certificate_repository.read(issuer_id) is None:
+                Logger.error(f"No such certificate found 'ID = {issuer_id}'.")
+                raise CertificateNotFoundException(issuer_id)
+        except DatabaseException:
+            Logger.error(f"No such certificate found 'ID = {issuer_id}'.")
+            raise CertificateNotFoundException(issuer_id)
 
     def set_certificate_revocation_status(self, id, status, reason="unspecified"):
         """
@@ -463 +487 @@
 
         return status
 
-
     def __get_crl_endpoint(self, ca_identifier: int) -> str:
         """
         Get URL address of CRL distribution endpoint based on
@@ -490 +513 @@
 
         return self.configuration.base_server_url + "/api/ocsp/" + str(ca_identifier)
 
-
 class RevocationReasonInvalidException(Exception):
     """
     Exception that denotes that the caller was trying to revoke