email-fetch-and-process 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a5a16792ae36fc27529004d4882e9a79e1df52efd8119c3bcb420ca1aae65288
-  data.tar.gz: 2a80e0b7e7e45db85af3472fac3cf41f36770a1c506c6fe81e490f7058db16ad
+  metadata.gz: 7e9ba4489394af5e199e4fb73c66da31339871c970d1dcfe01e4e97454774905
+  data.tar.gz: ec631ce7e5ddbdec7c4b27d6535cd9e819318dd7e8829ea93ad1af1e60fd76d4
 SHA512:
-  metadata.gz: 646c8bf3b023cf95deda4a8d391616aa9800e530ca0ba21d704b220b0d0fd95693bc7146bd8875078af2a3d33c600771c6f29f9868127a5118e14f2fc2e2234e
-  data.tar.gz: 0377a0b6ec94cc2e087ef2251ff03e2e54ac70e1b42e41b593eef34e1f4ba6c5c8527c4e3f7d6be913f93ec3dc1dd524b03db4ce240a8541446cabe6affc777d
+  metadata.gz: d966e8b64205b9ddd2eb7e64f09ec3187e0e495decee4fa0dd67fae0c3092200230cd11cf99c55abde47ae3584f9b8198ba6a14491485950a49f7dd5af1436bd
+  data.tar.gz: 20e24bae296292d8fcdec5cf21a715db0b16c032a0f83b7ec02e67bb1ca93725ec539075d5f613c7367a41abdd31010aeb2d4dd9e0e963f2469bc92ef4e3f2a0

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    email-fetch-and-process (0.1.1)
+    email-fetch-and-process (0.1.2)
 
 GEM
   remote: https://rubygems.org/

data/lib/email-fetch-and-process.rb

@@ -4,6 +4,7 @@ require 'net/imap'
 require 'fileutils'
 require 'mail'
 require 'time'
+require 'shellwords'
 require 'email-fetch-and-process/version'
 
 # Wrap up the logic to iterate through a bunch of fetch and handle
@@ -21,7 +22,8 @@ class EmailFetchAndProcess
         fetch: ['SUBJECT', ''],
         filename: '',
         action: 'echo FILEPATH',
-        subdirectory: nil
+        subdirectory: nil,
+        destination: '/tmp'
       }
     end
 
@@ -30,7 +32,7 @@ class EmailFetchAndProcess
     end
 
     def multiple_fetch_terms?
-      Array === @args[:fetch][0]
+      @args[:fetch][0].is_a?(Array)
     end
 
     def filename
@@ -44,6 +46,10 @@ class EmailFetchAndProcess
     def subdirectory
       @args[:subdirectory]
     end
+
+    def destination
+      @args[:destination]
+    end
   end
 
   attr_accessor :destination
@@ -65,7 +71,7 @@ class EmailFetchAndProcess
   end
 
   def imap_connection
-    imap = Net::IMAP.new(@args[:host], {:port => @args[:port], :ssl => {:verify_mode => OpenSSL::SSL::VERIFY_NONE}})
+    imap = Net::IMAP.new(@args[:host], port: @args[:port], ssl: { verify_mode: OpenSSL::SSL::VERIFY_NONE })
     imap.login(@args[:id], @args[:password])
     imap.examine(@args[:mailbox])
     imap
@@ -91,12 +97,13 @@ class EmailFetchAndProcess
 
         @body_index += 1
         attachment = part
-        attachment_path = File.expand_path(File.join(@destination, name))
-        attachment_path = nil unless attachment_path =~ /^#{@destination}/
-        next unless @destination != attachment_path
+        final_destination = job.destination || @destination
+        attachment_path = File.expand_path(File.join(final_destination, name))
+        attachment_path = nil unless attachment_path =~ /^#{final_destination}/
+        next unless final_destination != attachment_path
 
         if attachment && attachment_path
-          file_path = File.join([@destination, job.subdirectory, name].compact)
+          file_path = File.join([final_destination, job.subdirectory, name].compact)
           atch = attachment.body.to_s
           FileUtils.mkdir_p File.dirname(file_path) unless FileTest.exist? File.dirname(file_path)
           File.open(file_path, 'wb+') do |fh|
@@ -107,10 +114,10 @@ class EmailFetchAndProcess
           FileTest.exist?("#{file_path}.sha") &&
             File.open("#{file_path}.sha", 'r') { |fh| sha_old = fh.read.chomp }
           if sha_new != sha_old
-            command_to_run = job.action.gsub(/FILEPATH/, file_path)
-            if system(command_to_run)
-              File.open("#{file_path}.sha","w+") {|fh| fh.write sha_new }
-            end
+            command_to_run = job.action.gsub(/FILEPATH/, Shellwords.escape(file_path)).
+              gsub(/DESTINATION/, Shellwords.escape(final_destination))
+            system(command_to_run) &&
+              File.open("#{file_path}.sha", 'w+') { |fh| fh.write sha_new }
           end
         end
       end
@@ -122,13 +129,14 @@ class EmailFetchAndProcess
 
     jobs.each do |job|
       msg_ids = if job.multiple_fetch_terms?
-                  _ids = []
-                  job.fetch.each { |j| _ids += @imap.search(j) }
-                  _ids
+                  all_ids = []
+                  job.fetch.each { |j| all_ids += @imap.search(j) }
+                  all_ids
                 else
                   @imap.search(job.fetch)
                 end
       next if msg_ids.nil? || msg_ids.empty?
+
       begin
         msgs = @imap.fetch(msg_ids, %w[ENVELOPE RFC822])
         msg = msgs.max_by { |m| Time.parse(m['attr']['ENVELOPE'].date) }.attr['RFC822']

data/lib/email-fetch-and-process/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 class EmailFetchAndProcess
-  VERSION = '0.1.1'
+  VERSION = '0.1.2'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: email-fetch-and-process
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Kirk Haines