imap-filter 0.0.3 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2c399442d61849b86b6ab7f58a8d8e5e8e01566f
-  data.tar.gz: d19e15efc6dc6fdd1ad25a547fd2abac18cc1d95
+  metadata.gz: a240cdefbc038f384351e9905758f500052df1d1
+  data.tar.gz: 768b6df466f6af225b9b82a31e6c5c8cbdff2694
 SHA512:
-  metadata.gz: c8d3cc7800137f896ba21e5cb838b222379673a8d968060f1c530af56a572a2b75650bccba54336294725cefd6ddaeaa36dca1eebc694149ecc3877b43bf2bb4
-  data.tar.gz: 76210364367885058ab7bb22adf455579383d60d580f598b53758b4b7e1ee11174c478034c1f6d18b91cfdd4ddad64ac6619e19f2436f5cafd8f06dff20a0afb
+  metadata.gz: a08b499e9f1b7a400bb12b5966694b74f4f79205215e237e51add5defdf8dcb5224eb7b0ca67d763fe45109ecbbbde509d7e9029302f6b94c7005a4295ebe5b3
+  data.tar.gz: ffcba35a649f15287f9163a261be2eaca6f825b63231639cc00806d679be537f83e74ec3a83f931cef7d431cd60bfcf76dc3ef57d551665cd2d30babcdd6c4ce

data/.semver

@@ -1,5 +1,5 @@
 ---
 :major: 0
 :minor: 0
-:patch: 3
+:patch: 4
 :special: ''

data/.travis.yml

@@ -0,0 +1,9 @@
+language: ruby
+rvm:
+  - 2.0.0
+  - 2.2.0
+  - 2.2.1
+  - 2.3.0
+  - 2.3.1
+notifications:
+  irc: "irc.freenode.org#imap-filter"

data/Gemfile.lock

@@ -28,9 +28,9 @@ GEM
     highline (1.7.8)
     interception (0.5)
     json (1.8.3)
-    juwelier (2.1.2)
+    juwelier (2.1.3)
       builder
-      bundler (>= 1.0)
+      bundler (>= 1.13)
       git (>= 1.2.5)
       github_api
       highline (>= 1.6.15)
@@ -44,16 +44,14 @@ GEM
     multi_json (1.12.1)
     multi_xml (0.5.5)
     multipart-post (2.0.0)
-    nokogiri (1.6.8)
+    nokogiri (1.6.8.1)
       mini_portile2 (~> 2.1.0)
-      pkg-config (~> 1.1.7)
     oauth2 (1.2.0)
       faraday (>= 0.8, < 0.10)
       jwt (~> 1.0)
       multi_json (~> 1.3)
       multi_xml (~> 0.5)
       rack (>= 1.2, < 3)
-    pkg-config (1.1.7)
     pry (0.10.4)
       coderay (~> 1.1.0)
       method_source (~> 0.8.1)
@@ -121,4 +119,4 @@ DEPENDENCIES
   yard (~> 0)
 
 BUNDLED WITH
-   1.12.5
+   1.13.6

data/README.org

@@ -281,14 +281,17 @@
 *** Release Notes
     | Version | Breif                  | Description                                                                                                                    |
     |---------+------------------------+--------------------------------------------------------------------------------------------------------------------------------|
-    | v0.0.2  | Initial Release        |                                                                                                                                |
     | v0.0.3  | Fixed bug with timeout | On some lengthy remove copy/move operations, the source account can timeout. We do a simple NOOP to keep the connection alive. |
+    | v0.0.2  | Initial Release        |                                                                                                                                |
 
 *** Known Issues
-    |       Date | Description                                                    |
-    |------------+----------------------------------------------------------------|
-    | 2016-09-25 | Remote copy / moves of HTML-based email does not work cleanly. |
-    |            |                                                                |
+    |       Date | Description                                                                                                                                                                                                                           |
+    |------------+---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
+    | 2016-09-25 | Remote copy / moves of HTML-based email does not work cleanly.                                                                                                                                                                        |
+    | 2016-10-03 | Getting "Too many errors" ByeResponseError -- most likely from the errors of duplicate mailboxes (which is OK but some servers hate that). So I will have to check for the existence of the mailbox first before trying to create it. |
+    |            | Timeout errors still persist in some cases.                                                                                                                                                                                           |
+    |            | Need to capture all errors and provide human-readable responses, unless a certail verbosity level has been set.                                                                                                                       |
+    |            |                                                                                                                                                                                                                                       |
 
 ** Contributing to imap-filter
 

data/imap-filter.gemspec

@@ -2,19 +2,19 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Juwelier::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: imap-filter 0.0.3 ruby lib
+# stub: imap-filter 0.0.4 ruby lib
 
 Gem::Specification.new do |s|
-  s.name = "imap-filter"
-  s.version = "0.0.3"
+  s.name = "imap-filter".freeze
+  s.version = "0.0.4"
 
-  s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
-  s.require_paths = ["lib"]
-  s.authors = ["Fred Mitchell"]
-  s.date = "2016-10-01"
-  s.description = "\n  imap-filter is a Ruby implementation of an IMAP filtering application.\n  it can handle multiple IMAP accounts, and create IMAP folders automatically\n  where none exists.\n\n  The imap-filter DSL makes it easy to filter. You can also do \"dry-runs\"\n  to make sure what happens is what is expected."
-  s.email = "fred.mitchell@gmx.de"
-  s.executables = ["imap-filter"]
+  s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
+  s.require_paths = ["lib".freeze]
+  s.authors = ["Fred Mitchell".freeze]
+  s.date = "2016-11-05"
+  s.description = "\n  imap-filter is a Ruby implementation of an IMAP filtering application.\n  it can handle multiple IMAP accounts, and create IMAP folders automatically\n  where none exists.\n\n  The imap-filter DSL makes it easy to filter. You can also do \"dry-runs\"\n  to make sure what happens is what is expected.".freeze
+  s.email = "fred.mitchell@gmx.de".freeze
+  s.executables = ["imap-filter".freeze]
   s.extra_rdoc_files = [
     "LICENSE.txt",
     "README.org"
@@ -24,6 +24,7 @@ Gem::Specification.new do |s|
     ".rspec",
     ".ruby-version",
     ".semver",
+    ".travis.yml",
     "Gemfile",
     "Gemfile.lock",
     "LICENSE.txt",
@@ -41,73 +42,73 @@ Gem::Specification.new do |s|
     "spec/imap-filter_spec.rb",
     "spec/spec_helper.rb"
   ]
-  s.homepage = "http://github.com/flajann2/imap-filter"
-  s.licenses = ["MIT"]
-  s.required_ruby_version = Gem::Requirement.new(">= 2.2")
-  s.rubygems_version = "2.5.1"
-  s.summary = "IMAP Scriptable filter for one or multiple Email accounts."
+  s.homepage = "http://github.com/flajann2/imap-filter".freeze
+  s.licenses = ["MIT".freeze]
+  s.required_ruby_version = Gem::Requirement.new(">= 2.2".freeze)
+  s.rubygems_version = "2.6.7".freeze
+  s.summary = "IMAP Scriptable filter for one or multiple Email accounts.".freeze
 
   if s.respond_to? :specification_version then
     s.specification_version = 4
 
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
-      s.add_runtime_dependency(%q<semver>, ["~> 1"])
-      s.add_runtime_dependency(%q<awesome_print>, ["~> 1"])
-      s.add_runtime_dependency(%q<text-table>, ["~> 1"])
-      s.add_runtime_dependency(%q<thor>, ["~> 0"])
-      s.add_runtime_dependency(%q<colorize>, ["~> 0"])
-      s.add_runtime_dependency(%q<aspector>, ["~> 0"])
-      s.add_development_dependency(%q<rspec>, ["~> 2"])
-      s.add_development_dependency(%q<yard>, ["~> 0"])
-      s.add_development_dependency(%q<rdoc>, ["~> 3"])
-      s.add_development_dependency(%q<bundler>, ["~> 1"])
-      s.add_development_dependency(%q<juwelier>, ["~> 2"])
-      s.add_development_dependency(%q<simplecov>, ["~> 0"])
-      s.add_development_dependency(%q<pry>, ["~> 0"])
-      s.add_development_dependency(%q<pry-byebug>, ["~> 3"])
-      s.add_development_dependency(%q<pry-doc>, ["~> 0"])
-      s.add_development_dependency(%q<pry-remote>, ["~> 0"])
-      s.add_development_dependency(%q<pry-rescue>, ["~> 1"])
-      s.add_development_dependency(%q<pry-stack_explorer>, ["~> 0"])
+      s.add_runtime_dependency(%q<semver>.freeze, ["~> 1"])
+      s.add_runtime_dependency(%q<awesome_print>.freeze, ["~> 1"])
+      s.add_runtime_dependency(%q<text-table>.freeze, ["~> 1"])
+      s.add_runtime_dependency(%q<thor>.freeze, ["~> 0"])
+      s.add_runtime_dependency(%q<colorize>.freeze, ["~> 0"])
+      s.add_runtime_dependency(%q<aspector>.freeze, ["~> 0"])
+      s.add_development_dependency(%q<rspec>.freeze, ["~> 2"])
+      s.add_development_dependency(%q<yard>.freeze, ["~> 0"])
+      s.add_development_dependency(%q<rdoc>.freeze, ["~> 3"])
+      s.add_development_dependency(%q<bundler>.freeze, ["~> 1"])
+      s.add_development_dependency(%q<juwelier>.freeze, ["~> 2"])
+      s.add_development_dependency(%q<simplecov>.freeze, ["~> 0"])
+      s.add_development_dependency(%q<pry>.freeze, ["~> 0"])
+      s.add_development_dependency(%q<pry-byebug>.freeze, ["~> 3"])
+      s.add_development_dependency(%q<pry-doc>.freeze, ["~> 0"])
+      s.add_development_dependency(%q<pry-remote>.freeze, ["~> 0"])
+      s.add_development_dependency(%q<pry-rescue>.freeze, ["~> 1"])
+      s.add_development_dependency(%q<pry-stack_explorer>.freeze, ["~> 0"])
     else
-      s.add_dependency(%q<semver>, ["~> 1"])
-      s.add_dependency(%q<awesome_print>, ["~> 1"])
-      s.add_dependency(%q<text-table>, ["~> 1"])
-      s.add_dependency(%q<thor>, ["~> 0"])
-      s.add_dependency(%q<colorize>, ["~> 0"])
-      s.add_dependency(%q<aspector>, ["~> 0"])
-      s.add_dependency(%q<rspec>, ["~> 2"])
-      s.add_dependency(%q<yard>, ["~> 0"])
-      s.add_dependency(%q<rdoc>, ["~> 3"])
-      s.add_dependency(%q<bundler>, ["~> 1"])
-      s.add_dependency(%q<juwelier>, ["~> 2"])
-      s.add_dependency(%q<simplecov>, ["~> 0"])
-      s.add_dependency(%q<pry>, ["~> 0"])
-      s.add_dependency(%q<pry-byebug>, ["~> 3"])
-      s.add_dependency(%q<pry-doc>, ["~> 0"])
-      s.add_dependency(%q<pry-remote>, ["~> 0"])
-      s.add_dependency(%q<pry-rescue>, ["~> 1"])
-      s.add_dependency(%q<pry-stack_explorer>, ["~> 0"])
+      s.add_dependency(%q<semver>.freeze, ["~> 1"])
+      s.add_dependency(%q<awesome_print>.freeze, ["~> 1"])
+      s.add_dependency(%q<text-table>.freeze, ["~> 1"])
+      s.add_dependency(%q<thor>.freeze, ["~> 0"])
+      s.add_dependency(%q<colorize>.freeze, ["~> 0"])
+      s.add_dependency(%q<aspector>.freeze, ["~> 0"])
+      s.add_dependency(%q<rspec>.freeze, ["~> 2"])
+      s.add_dependency(%q<yard>.freeze, ["~> 0"])
+      s.add_dependency(%q<rdoc>.freeze, ["~> 3"])
+      s.add_dependency(%q<bundler>.freeze, ["~> 1"])
+      s.add_dependency(%q<juwelier>.freeze, ["~> 2"])
+      s.add_dependency(%q<simplecov>.freeze, ["~> 0"])
+      s.add_dependency(%q<pry>.freeze, ["~> 0"])
+      s.add_dependency(%q<pry-byebug>.freeze, ["~> 3"])
+      s.add_dependency(%q<pry-doc>.freeze, ["~> 0"])
+      s.add_dependency(%q<pry-remote>.freeze, ["~> 0"])
+      s.add_dependency(%q<pry-rescue>.freeze, ["~> 1"])
+      s.add_dependency(%q<pry-stack_explorer>.freeze, ["~> 0"])
     end
   else
-    s.add_dependency(%q<semver>, ["~> 1"])
-    s.add_dependency(%q<awesome_print>, ["~> 1"])
-    s.add_dependency(%q<text-table>, ["~> 1"])
-    s.add_dependency(%q<thor>, ["~> 0"])
-    s.add_dependency(%q<colorize>, ["~> 0"])
-    s.add_dependency(%q<aspector>, ["~> 0"])
-    s.add_dependency(%q<rspec>, ["~> 2"])
-    s.add_dependency(%q<yard>, ["~> 0"])
-    s.add_dependency(%q<rdoc>, ["~> 3"])
-    s.add_dependency(%q<bundler>, ["~> 1"])
-    s.add_dependency(%q<juwelier>, ["~> 2"])
-    s.add_dependency(%q<simplecov>, ["~> 0"])
-    s.add_dependency(%q<pry>, ["~> 0"])
-    s.add_dependency(%q<pry-byebug>, ["~> 3"])
-    s.add_dependency(%q<pry-doc>, ["~> 0"])
-    s.add_dependency(%q<pry-remote>, ["~> 0"])
-    s.add_dependency(%q<pry-rescue>, ["~> 1"])
-    s.add_dependency(%q<pry-stack_explorer>, ["~> 0"])
+    s.add_dependency(%q<semver>.freeze, ["~> 1"])
+    s.add_dependency(%q<awesome_print>.freeze, ["~> 1"])
+    s.add_dependency(%q<text-table>.freeze, ["~> 1"])
+    s.add_dependency(%q<thor>.freeze, ["~> 0"])
+    s.add_dependency(%q<colorize>.freeze, ["~> 0"])
+    s.add_dependency(%q<aspector>.freeze, ["~> 0"])
+    s.add_dependency(%q<rspec>.freeze, ["~> 2"])
+    s.add_dependency(%q<yard>.freeze, ["~> 0"])
+    s.add_dependency(%q<rdoc>.freeze, ["~> 3"])
+    s.add_dependency(%q<bundler>.freeze, ["~> 1"])
+    s.add_dependency(%q<juwelier>.freeze, ["~> 2"])
+    s.add_dependency(%q<simplecov>.freeze, ["~> 0"])
+    s.add_dependency(%q<pry>.freeze, ["~> 0"])
+    s.add_dependency(%q<pry-byebug>.freeze, ["~> 3"])
+    s.add_dependency(%q<pry-doc>.freeze, ["~> 0"])
+    s.add_dependency(%q<pry-remote>.freeze, ["~> 0"])
+    s.add_dependency(%q<pry-rescue>.freeze, ["~> 1"])
+    s.add_dependency(%q<pry-stack_explorer>.freeze, ["~> 0"])
   end
 end
 

data/lib/imap-filter/cli.rb

@@ -19,6 +19,9 @@ module ImapFilter
         puts "script %s " % [script] if _options[:verbose] >= 1
         load script, true
       end
+
+      def list
+      end
     end
   end
 end

data/lib/imap-filter/dsl.rb

@@ -62,7 +62,7 @@ module ImapFilter
         super
         @name = name
         instance_eval( &block )
-        _accounts[name] = self        
+        _accounts[name] = self      
       end
 
       def to_s
@@ -81,7 +81,7 @@ module ImapFilter
 
         print "\n    *** auth #{userid} pass #{pass}...".light_cyan unless _options[:verbose] < 2
         imap.authenticate(auth_type, userid, pass)
-        @delim = imap.list('', '').first.delim
+        @delim = imap.list('', '').first.delim        
       end
 
       def _close_connection

data/lib/imap-filter/functionality.rb

@@ -3,7 +3,55 @@ module ImapFilter
   module Functionality
     include Forwardable
     include ImapFilter::DSL
+    @@facc_list = nil
+    
+    def self._functional_accounts
+      @@facc_list ||= _accounts.map { |name, acc|
+        [name, FunctAccount.new( acc )]
+      }.to_h
+    end
+    
+    class FunctAccount
+      extend Forwardable
+      
+      attr :dacc
+      
+      def_delegators :@dacc, :name, :userid, :pass,
+                     :fqdn, :use_ssl, :use_port,
+                     :auth_type, :imap, :delim, :to_s,
+                     :_open_connection, :_close_connection
 
+      def initialize acc
+        @dacc = acc
+      end
+
+      def mbox_list
+        @mbox_list ||= @dacc.imap.list('', '*')
+                     .map { |m| [m['name'], m['attr']] }
+                     .map { |mbox, attr|
+          begin
+            [mbox,
+             [@dacc.imap.status(mbox, STATUS.values)
+               .map{ |k, v| "#{ISTAT[k]}:#{v}" }
+               .join(' '),
+              attr]]
+          rescue
+            nil
+          end }.compact.to_h
+      end
+
+      def member? mbox
+        mbox_list.member? mbox
+      end
+
+      def ensure_mailbox mbox
+        unless member? mbox
+          @dacc.imap.create mbox
+          @mbox_list[mbox] = [:new_mailbox, :new_mailbox]
+        end
+      end
+    end
+    
     FULL ='(UID RFC822.SIZE ENVELOPE BODY.PEEK[TEXT])'
     BODYTEXT = 'BODY[TEXT]'
     SUBJECTPEEKLIST = '(UID RFC822.SIZE BODY.PEEK[HEADER.FIELDS (SUBJECT)])'
@@ -27,7 +75,7 @@ module ImapFilter
         a, b = ambox.split ':'
         a, b = [nil, a] if b.nil?
         a = nil if a == ''
-        acc = a.nil? ? default_account : _accounts[a.to_sym]
+        acc = FunctAccount.new( a.nil? ? default_account : _accounts[a.to_sym] )
         mbox = b
         [acc, mbox]
       end
@@ -40,9 +88,8 @@ module ImapFilter
 
       def ensure_mailbox account, mailbox
         begin
-          account.imap.create mailbox
+          account.ensure_mailbox mailbox
         rescue Net::IMAP::NoResponseError => e
-          # we ignore this because it -- probably -- means the mailbox already exists.
           puts "  *** ignored mailbox error: #{e}".red unless _options[:verbose] < 1
         end
       end

data/lib/imap-filter/imap-filter.rb

@@ -24,30 +24,16 @@ module ImapFilter
     end
 
     # List all mboxes of given account and their statuses
-    def self.list_mboxes account
-      account.imap.list('', '*')
-        .map { |m| [m['name'], m['attr']] }
-        .map { |mbox, attr|
-        begin
-          [mbox,
-           account.imap.status(mbox, STATUS.values)
-             .map{ |k, v| "#{ISTAT[k]}:#{v}" }
-             .join(' '),
-           attr]
-        rescue
-          nil
-        end }
-        .compact
-    end
     
     def self.login_imap_accounts test: false
       puts "====== #{test ? 'Test' : 'Login'} Accounts".light_yellow
-      _accounts.each do |name, account|
+      _functional_accounts.each do |name, account|
         print "  #{name}...".light_white
         begin
           account._open_connection
-          puts "SUCCESS, delim #{account.delim}".light_green
-          list_mboxes(account).each do |mbox, stat, attr|
+          puts "SUCCESS, delim #{account.delim}".light_green          
+          
+          account.mbox_list.each do |mbox, (stat, attr)|
             print "  #{mbox}".light_blue
             print " #{stat}".light_red
             puts " #{attr}".light_cyan

data/lib/imap-filter/monkeypatches.rb

@@ -46,4 +46,3 @@ end
 class IMAPHooks < Aspector::Base
   default private_methods: true
 end
-

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: imap-filter
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors:
 - Fred Mitchell
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-10-01 00:00:00.000000000 Z
+date: 2016-11-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: semver
@@ -282,6 +282,7 @@ files:
 - ".rspec"
 - ".ruby-version"
 - ".semver"
+- ".travis.yml"
 - Gemfile
 - Gemfile.lock
 - LICENSE.txt
@@ -318,7 +319,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.6.7
 signing_key: 
 specification_version: 4
 summary: IMAP Scriptable filter for one or multiple Email accounts.