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

caerp_sign_pdf/public.py

@@ -4,11 +4,12 @@ import hashlib
 
 from io import BytesIO
 from pyhanko.pdf_utils.incremental_writer import IncrementalPdfFileWriter
+from pyhanko.pdf_utils.xref import ObjectHeaderReadError
 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 +19,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 +98,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 +118,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 +137,22 @@ 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)
+        except ObjectHeaderReadError as error:
+            logger.error(f"Error while reading Xref header : {error}")
+            raise HTTPUnsupportedMediaType(ERROR_PDF_FORMAT_MSG)
+
+        # 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.4.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
-Name: caerp-sign-pdf
-Version: 2024.1.2
+Name: caerp_sign_pdf
+Version: 2024.1.4
 Summary: caerp_sign_pdf
 Home-page: https://framagit.org/caerp/caerp_sign_pdf
 Author: Kilya
@@ -14,7 +14,7 @@ Classifier: Topic :: Internet :: WWW/HTTP
 Classifier: Topic :: Internet :: WWW/HTTP :: WSGI :: Application
 Description-Content-Type: text/x-rst
 License-File: LICENSE.txt
-Requires-Dist: pyHanko[image-support,opentype,pkcs11,xmp] (==0.25.0)
+Requires-Dist: pyHanko[image-support,opentype,pkcs11,xmp] ==0.25.1
 
 CAERP Sign PDF
 ======================================================

caerp_sign_pdf-2024.1.4.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=Ocb7dULKWAz_Jhumb9SWOvAoPQ8cY1Jy3Aa3s3_tqV8,6090
+caerp_sign_pdf-2024.1.4.dist-info/LICENSE.txt,sha256=jOtLnuWt7d5Hsx6XXB2QxzrSe2sWWh3NgMfFRetluQM,35147
+caerp_sign_pdf-2024.1.4.dist-info/METADATA,sha256=9JAUmLKLwZHRGuOx7pZuvtLlowohjh9vWXTIUlhkMi0,2399
+caerp_sign_pdf-2024.1.4.dist-info/WHEEL,sha256=yQN5g4mg4AybRjkgi-9yy4iQEFibGQmlz78Pik5Or-A,92
+caerp_sign_pdf-2024.1.4.dist-info/top_level.txt,sha256=kNSdxjlamRUTFu_IvJHzd1pJIv12fmuWotrcJ2sRgEw,15
+caerp_sign_pdf-2024.1.4.dist-info/RECORD,,

{caerp_sign_pdf-2024.1.2.dist-info → caerp_sign_pdf-2024.1.4.dist-info}/WHEEL

@@ -1,5 +1,5 @@
 Wheel-Version: 1.0
-Generator: bdist_wheel (0.37.1)
+Generator: bdist_wheel (0.41.2)
 Root-Is-Purelib: true
 Tag: py3-none-any
 

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,,