From 4d05881f37683014a800f9d94b5d83f6089e0005 Mon Sep 17 00:00:00 2001
From: Antonio Espinosa <antespi@gmail.com>
Date: Wed, 27 Jan 2016 12:45:00 +0100
Subject: [PATCH] [IMP] Define jPdfSign max heap size

---
 report_qweb_signer/models/report.py | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/report_qweb_signer/models/report.py b/report_qweb_signer/models/report.py
index 850650ff..8245cd39 100644
--- a/report_qweb_signer/models/report.py
+++ b/report_qweb_signer/models/report.py
@@ -116,8 +116,8 @@ class Report(models.Model):
             })
         except AccessError:
             raise Warning(
-                _('Saving signed report (PDF):') + '\n',
-                _('You do not have enought access rights to save attachments'))
+                _('Saving signed report (PDF): '
+                  'You do not have enought access rights to save attachments'))
         else:
             _logger.info(
                 "The signed PDF document '%s' is now saved in the database",
@@ -126,7 +126,9 @@ class Report(models.Model):
 
     def _signer_bin(self, opts):
         me = os.path.dirname(__file__)
-        return 'java -jar {}/../static/jar/jPdfSign.jar '.format(me) + opts
+        java_bin = 'java -jar -Xms4M -Xmx4M'
+        jar = '{}/../static/jar/jPdfSign.jar'.format(me)
+        return '%s %s %s' % (java_bin, jar, opts)
 
     def pdf_sign(self, pdf, certificate):
         pdfsigned = pdf + '.signed.pdf'
@@ -134,8 +136,8 @@ class Report(models.Model):
         passwd = _normalize_filepath(certificate.password_file)
         if not (p12 and passwd):
             raise Warning(
-                _('Signing report (PDF)'),
-                _('Certificate or password file not found'))
+                _('Signing report (PDF): '
+                  'Certificate or password file not found'))
         signer_opts = '"%s" "%s" "%s" "%s"' % (p12, pdf, pdfsigned, passwd)
         signer = self._signer_bin(signer_opts)
         process = subprocess.Popen(
@@ -143,9 +145,9 @@ class Report(models.Model):
         out, err = process.communicate()
         if process.returncode:
             raise Warning(
-                _('Signing report (PDF)'),
-                _('jPdfSign failed (error code: %s). '
-                  'Message: %s') % (str(process.returncode), err))
+                _('Signing report (PDF): jPdfSign failed (error code: %s). '
+                  'Message: %s. Output: %s') %
+                (process.returncode, err, out))
         return pdfsigned
 
     @api.v7