caerp-sign-pdf 2024.1.2__py3-none-any.whl → 2024.1.3__py3-none-any.whl

caerp_sign_pdf/public.py

@@ -7,8 +7,8 @@ from pyhanko.pdf_utils.incremental_writer import IncrementalPdfFileWriter
 from pyhanko.sign import fields, signers
 from pyhanko.sign.general import SigningError
 from pyhanko.stamp import TextStampStyle
-from PyPDF4.pdf import PdfFileReader, PdfFileWriter, utils
-from pyramid.httpexceptions import HTTPInternalServerError
+from PyPDF4.pdf import PdfFileReader, PdfFileWriter
+from pyramid.httpexceptions import HTTPUnsupportedMediaType
 
 from caerp.views.files.controller import FileData
 
@@ -18,6 +18,14 @@ from .models import PDFSignatureHistory
 logger = logging.getLogger(f"caerp.{__name__}")
 
 
+ERROR_LOAD_CERTIFICATE_MSG = "Impossible de charger le certificat à utiliser \
+pour signer numériquement le fichier. Veuillez contacter le support."
+
+ERROR_PDF_FORMAT_MSG = "Le fichier PDF ne peut pas être signé numériquement \
+car son format n'est pas correct. Vous pouvez essayer de l'imprimer dans un \
+nouveau fichier PDF avant de le recharger."
+
+
 class SignPDFService(object):
     def __init__(self, context, request):
         self.context = context
@@ -89,11 +97,11 @@ class SignPDFService(object):
             passphrase=self._get_certificate_passphrase().encode(),
         )
         if not signer:
-            raise Exception(
+            logger.error(
                 "Unable to load certificate for signing PDF files (check \
-'caerp.sign_certificate_path' and 'caerp.sign_certificate_passphrase' \
-in config)"
+'caerp.sign_certificate_path' and 'caerp.sign_certificate_passphrase' in config)"
             )
+            raise Exception(ERROR_LOAD_CERTIFICATE_MSG)
 
         # Prepare stamp (if needed)
         stamp = None
@@ -109,21 +117,18 @@ in config)"
                 border_width=0,
             )
 
-        # Clean PDF before signing
+        # Open original PDF
         try:
             pdf_data.data = self._get_safe_pdf(pdf_data.data)
+            pdf_writer = IncrementalPdfFileWriter(pdf_data.data, strict=False)
         except Exception as error:
-            logger.error(f"Unknown error while cleaning original PDF file : {error}")
-            raise HTTPInternalServerError(
-                "Le fichier PDF ne peut pas être traité car son format n'est pas \
-correct. Vous pouvez essayer de le réimprimer dans un nouveau fichier PDF."
-            )
+            logger.error(f"Error while opening original PDF file : {error}")
+            raise HTTPUnsupportedMediaType(ERROR_PDF_FORMAT_MSG)
 
-        # Sign PDF
-        w = IncrementalPdfFileWriter(pdf_data.data, strict=False)
+        # Add signature field on the PDF
         try:
             fields.append_signature_field(
-                w,
+                pdf_writer,
                 sig_field_spec=fields.SigFieldSpec(
                     "SignatureCAERP", box=(445, 800, 580, 830)
                 ),
@@ -131,13 +136,19 @@ correct. Vous pouvez essayer de le réimprimer dans un nouveau fichier PDF."
         except SigningError:
             logger.warn("File is already signed, nothing to do")
             return True
-        meta = signers.PdfSignatureMetadata(field_name="SignatureCAERP")
-        pdf_signer = signers.PdfSigner(
-            meta,
-            signer=signer,
-            stamp_style=stamp,
-        )
-        pdf_data.data = pdf_signer.sign_pdf(w)
+
+        # Sign PDF
+        try:
+            meta = signers.PdfSignatureMetadata(field_name="SignatureCAERP")
+            pdf_signer = signers.PdfSigner(
+                meta,
+                signer=signer,
+                stamp_style=stamp,
+            )
+            pdf_data.data = pdf_signer.sign_pdf(pdf_writer)
+        except Exception as error:
+            logger.error(f"Error while signing PDF file : {error}")
+            raise HTTPUnsupportedMediaType(ERROR_PDF_FORMAT_MSG)
         logger.info(
             "File '{}' signed successfully by {} !".format(
                 pdf_data.name, signer.subject_name

{caerp_sign_pdf-2024.1.2.dist-info → caerp_sign_pdf-2024.1.3.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: caerp-sign-pdf
-Version: 2024.1.2
+Version: 2024.1.3
 Summary: caerp_sign_pdf
 Home-page: https://framagit.org/caerp/caerp_sign_pdf
 Author: Kilya

caerp_sign_pdf-2024.1.3.dist-info/RECORD

@@ -0,0 +1,8 @@
+caerp_sign_pdf/__init__.py,sha256=vT-YO-kLKOqDIFKyVo_YGpLvZMZNNonAksW7dVWaScs,192
+caerp_sign_pdf/models.py,sha256=R_-jV-hO0Pm4_Rd1evyNerx8vuXELhMecNODdI598B4,592
+caerp_sign_pdf/public.py,sha256=jZ7BZkOSQVBZ6loebSwduEGcydthRx06rmtO891n14A,5850
+caerp_sign_pdf-2024.1.3.dist-info/LICENSE.txt,sha256=jOtLnuWt7d5Hsx6XXB2QxzrSe2sWWh3NgMfFRetluQM,35147
+caerp_sign_pdf-2024.1.3.dist-info/METADATA,sha256=VNsy8FY9x8hz6OMp4opl2w8nnFO_cbzSEd9nvECU-uA,2401
+caerp_sign_pdf-2024.1.3.dist-info/WHEEL,sha256=G16H4A3IeoQmnOrYV4ueZGKSjhipXx8zc8nu9FGlvMA,92
+caerp_sign_pdf-2024.1.3.dist-info/top_level.txt,sha256=kNSdxjlamRUTFu_IvJHzd1pJIv12fmuWotrcJ2sRgEw,15
+caerp_sign_pdf-2024.1.3.dist-info/RECORD,,

caerp_sign_pdf-2024.1.2.dist-info/RECORD

@@ -1,8 +0,0 @@
-caerp_sign_pdf/__init__.py,sha256=vT-YO-kLKOqDIFKyVo_YGpLvZMZNNonAksW7dVWaScs,192
-caerp_sign_pdf/models.py,sha256=R_-jV-hO0Pm4_Rd1evyNerx8vuXELhMecNODdI598B4,592
-caerp_sign_pdf/public.py,sha256=VUMCfPACZ8xaqpldl6ly2BlWUnaHFExXhLIrBxttq9Q,5336
-caerp_sign_pdf-2024.1.2.dist-info/LICENSE.txt,sha256=jOtLnuWt7d5Hsx6XXB2QxzrSe2sWWh3NgMfFRetluQM,35147
-caerp_sign_pdf-2024.1.2.dist-info/METADATA,sha256=ZQCLNLiq0arJE3So7RfqIjb9h84M4S7x1fa4I17qBEY,2401
-caerp_sign_pdf-2024.1.2.dist-info/WHEEL,sha256=G16H4A3IeoQmnOrYV4ueZGKSjhipXx8zc8nu9FGlvMA,92
-caerp_sign_pdf-2024.1.2.dist-info/top_level.txt,sha256=kNSdxjlamRUTFu_IvJHzd1pJIv12fmuWotrcJ2sRgEw,15
-caerp_sign_pdf-2024.1.2.dist-info/RECORD,,