mailqun 0.1.3 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a6fb93beeb19169d7babb25af4ba82437ddb4128
-  data.tar.gz: 74770a11fceebcd33720dfa8eef25885458de5fd
+  metadata.gz: aa92a4457288ba1a54a43c6d9921aa5cf119e642
+  data.tar.gz: 262bd8e2701d04312974ee84fac86f716d538553
 SHA512:
-  metadata.gz: 8c12b80fcba5d1082ef62a6a48b4d92b62f72215d45b475b400e2402c774d9946e33057819bf0e2bc4451a8000e585dab787fcb7767c9f5f2d61c2039d705956
-  data.tar.gz: 62bf63ee66d3bcd2542b8a2986cfbac98bd57e6e0963ee3f54184ea368fb2983064bd158955563f4acf9eed8daba8c8f763ca31fa1590903380cafe71bb51419
+  metadata.gz: 75898450897035e70684127c56e576d6ba16521ab043bcd675760fed4b71465e9ca4abbc81364f8d215f762d4dc5f86802bfeef686d98e5ae711bd2d3f10ef68
+  data.tar.gz: 4edafe4f40b695d4ff1f78dd845dcead9e092f0bb4b7645957ccac4443e594c0b9a98b642dd09d28933f50c90990c52f554c8a1a079876038ab0f2beab3eaa6b

data/mailqun

@@ -3,27 +3,14 @@ require 'net/http'
 require 'optparse'
 require 'ostruct'
 require 'json'
-
-begin
-  require 'colorize'
-rescue LoadError
-  class String
-    def method_missing(name)
-      if ['green', 'red'].include? name.to_s
-        self
-      else
-        super
-      end
-    end
-  end
-end
+require 'colorize'
 
 def load_env
   @domain = ENV['MAILGUN_DOMAIN']
   @secret = ENV['MAILGUN_SECRET']
 
-  abort("Error: Set MAILGUN_DOMAIN") if @domain.nil?
-  abort("Error: Set MAILGUN_SECRET") if @secret.nil?
+  abort('Error: Set MAILGUN_DOMAIN') if @domain.nil?
+  abort('Error: Set MAILGUN_SECRET') if @secret.nil?
 end
 
 def load_uri
@@ -36,29 +23,29 @@ def load_uri
     query = URI.encode_www_form(@options.to_h)
     url = "https://api.mailgun.net/v3/#{@domain}/#{@query_type}?#{query}"
   end
-  return URI(url)
+  URI(url)
 end
 
 def request uri
   response = nil
-  Net::HTTP.start(uri.host, uri.port, :use_ssl => uri.scheme == 'https') do |http|
+  Net::HTTP.start(uri.host, uri.port, use_ssl: uri.scheme == 'https') do |http|
     klass = Object.const_get("Net::HTTP::#{@verb.capitalize}")
     request = klass.new uri.request_uri
     request.basic_auth 'api', @secret
     response = http.request request
   end
-  return response
+  response
 end
 
-def get_json_response
+def build_json_response
   response = request load_uri
-  unless response.kind_of? Net::HTTPSuccess
+  unless response.is_a? Net::HTTPSuccess
     abort("Error: Response code was #{response.code}\n#{response.body}")
   end
   JSON.parse(response.body)
 end
 
-def get_filename filename
+def build_filename filename
   filename.gsub!(/^.*(\\|\/)/, '')
   filename.gsub!(/[^0-9A-Za-z.\-]/, '_')
   "#{filename.strip}_#{Time.now.to_i}.html"
@@ -66,29 +53,37 @@ end
 
 def pretty_print_event_item item
   event = item['event'].upcase
-  pretty_line = "#{Time.at(item['timestamp'])}: [#{event}]"
-  pretty_line = "#{pretty_line}\t[#{item['tags'].join(',')}]";
-  pretty_line = "#{pretty_line} #{item['recipient']}"
-
+  pretty_line = pretty_line_prefix item
   if event == 'FAILED'
-    error_description = item['delivery-status']['description']
-    error_description = item['delivery-status']['message'] if error_description.empty?
-    pretty_line = "#{pretty_line}\t- #{item['delivery-status']['code']}"\
-                  " - #{error_description}"
-
-    puts "#{pretty_line}".red
+    puts pretty_line_failed(pretty_line, item).to_s.red
   elsif ['DELIVERED', 'OPENED'].include? event
-    puts "#{pretty_line}".green
+    puts pretty_line.to_s.green
   else
-    puts "#{pretty_line}"
+    puts pretty_line.to_s
   end
 end
 
+def pretty_line_failed pretty_line, item
+  error_description = [
+    item['delivery-status']['description'],
+    item['delivery-status']['message']
+  ].find { |m| !m.empty? }[0..70].gsub(/\s\w+$/, '...')
+  "#{pretty_line}\t- #{item['delivery-status']['code']} - #{error_description}"
+end
+
+def pretty_line_prefix item
+  "#{Time.at(item['timestamp'])}: " \
+  "[#{item['event'].upcase}]" \
+  "\t" \
+  "[#{item['tags'].join(',')}] " \
+  "#{item['recipient']}"
+end
+
 def pretty_print_bounce_item item
   pretty_line = "#{item['created_at']}: #{item['address']}"
-  pretty_line = "#{pretty_line}\t- #{item['error'].red}";
+  pretty_line = "#{pretty_line}\t- #{item['error'].red}"
 
-  puts "#{pretty_line}"
+  puts pretty_line.to_s
 end
 
 def pretty_print_item item
@@ -115,8 +110,8 @@ def handle_fetch json_response
 end
 
 def handle_view json_response
-  filename = get_filename json_response['subject']
-  open("#{filename}", 'w') do |f|
+  filename = build_filename json_response['subject']
+  File.open(filename.to_s, 'w') do |f|
     f.puts json_response['body-html']
   end
   system "open #{filename}"
@@ -137,74 +132,72 @@ end
 @verb = 'GET'
 
 OptionParser.new do |opts|
-  opts.banner = "Usage: mailqun [options]"
+  opts.banner = 'Usage: mailqun [options]'
 
-  opts.separator ""
-  opts.separator "Common options:"
+  opts.separator ''
+  opts.separator 'Common options:'
 
-  opts.on("-f [FROM]", "--from [FROM]", String,
-          "Address of the sender") do |f|
+  opts.on('-f [FROM]', '--from [FROM]', String,
+          'Address of the sender') do |f|
     @options.from = f
   end
 
-  opts.on("-t [TO]", "--to [TO]", String,
-          "Address of the receiver") do |t|
+  opts.on('-t [TO]', '--to [TO]', String,
+          'Address of the receiver') do |t|
     @options.to = t
   end
 
-  opts.on("-s [SUBJECT]", "--subject [SUBJECT]", String,
-          "Subject of mail") do |s|
+  opts.on('-s [SUBJECT]', '--subject [SUBJECT]', String,
+          'Subject of mail') do |s|
     @options.subject = s
   end
 
-  opts.on("-l [LIMIT]", "--limit [LIMIT]", Integer,
-          "Limit on number of results") do |l|
+  opts.on('-l [LIMIT]', '--limit [LIMIT]', Integer,
+          'Limit on number of results') do |l|
     @options.limit = l
   end
 
-  opts.on("--tags [TAG]", String,
-          "Value in X-Mailgun-Tag header") do |t|
+  opts.on('--tags [TAG]', String,
+          'Value in X-Mailgun-Tag header') do |t|
     @options.tags = t
   end
 
-  opts.on("-e [EVENT]", "--event [EVENT]", String,
-          "Event being queried") do |e|
+  opts.on('-e [EVENT]', '--event [EVENT]', String,
+          'Event being queried') do |e|
     @options.event = e
   end
 
-  opts.on("--raw",
-          "Gives raw json response, instead of summary") do |e|
+  opts.on('--raw',
+          'Gives raw json response, instead of summary') do |_e|
     @options.pretty = false
   end
 
-  opts.separator ""
-  opts.separator "Bounce options:"
+  opts.separator ''
+  opts.separator 'Bounce options:'
 
-  opts.on("--bounces",
-          "Add this option to query bounces instead of events") do |e|
+  opts.on('--bounces',
+          'Add this option to query bounces instead of events') do |_e|
     @query_type = 'bounces'
   end
 
-  opts.on("-a [ADDRESS]", "--address [ADDRESS]", String,
-          "Bounced address, used with --bounces") do |a|
-    if @query_type == 'bounces'
-      @address = a
-    end
+  opts.on('-a [ADDRESS]', '--address [ADDRESS]', String,
+          'Bounced address, used with --bounces') do |a|
+    @address = a if @query_type == 'bounces'
   end
 
-  opts.on("--delete [ADDRESS]", String,
-          "Delete address from bounces, used with --bounces") do |a|
+  opts.on('--delete [ADDRESS]', String,
+          'Delete address from bounces, used with --bounces') do |a|
     if @query_type == 'bounces'
       @address = a
       @verb = 'DELETE'
     end
   end
 
-  opts.separator ""
-  opts.separator "View options:"
+  opts.separator ''
+  opts.separator 'View options:'
 
-  opts.on("--view-body [URL]",
-          "Mailgun storage URL to fetch email body from") do |u|
+  opts.on('--view-body [URL]',
+          'Mailgun storage URL to fetch email body from') do |u|
     @url = u
     @query_type = 'view'
   end
@@ -212,6 +205,6 @@ end.parse!
 
 load_env
 
-json_response = get_json_response
+json_response = build_json_response
 
 build_output json_response

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mailqun
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.2.0
 platform: ruby
 authors:
 - Harman Singh