You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@whimsical.apache.org by ru...@apache.org on 2018/10/31 12:57:58 UTC

[whimsy] 01/01: switch to the pip3 version of img2pdf

This is an automated email from the ASF dual-hosted git repository.

rubys pushed a commit to branch pip3-img2pdf
in repository https://gitbox.apache.org/repos/asf/whimsy.git

commit abf9b3c12b47f26933bb2cb1c0bdd43e560053b7
Author: Sam Ruby <ru...@intertwingly.net>
AuthorDate: Wed Oct 31 08:56:15 2018 -0400

    switch to the pip3 version of img2pdf
---
 www/secretary/workbench/img2pdf              | 7 +++++++
 www/secretary/workbench/models/attachment.rb | 3 ++-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/www/secretary/workbench/img2pdf b/www/secretary/workbench/img2pdf
new file mode 100755
index 0000000..fe48b71
--- /dev/null
+++ b/www/secretary/workbench/img2pdf
@@ -0,0 +1,7 @@
+#!/usr/bin/python3
+
+# Alternate executable wrapper for img2pdf, created because the packaged
+# version of img2pdf on Ubuntu 16.04 contains a back-level version
+
+import img2pdf
+img2pdf.main()
diff --git a/www/secretary/workbench/models/attachment.rb b/www/secretary/workbench/models/attachment.rb
index edeea31..b95771b 100644
--- a/www/secretary/workbench/models/attachment.rb
+++ b/www/secretary/workbench/models/attachment.rb
@@ -61,7 +61,8 @@ class Attachment
 
     if IMAGE_TYPES.include? ext or content_type.start_with? 'image/'
       pdf = SafeTempFile.new([safe_name, '.pdf'])
-      system 'img2pdf', '--output', pdf.path, file.path
+      img2pdf = File.expand_path('../img2pdf', __dir__.untaint).untaint
+      system img2pdf, '--output', pdf.path, file.path
       file.unlink
       return pdf
     end