griddler-sendgrid 0.0.1 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: be5d9a9db3c7daa2c8becdb06c09410bfeea98e1
-  data.tar.gz: bf3746c3b5e995637e8e54c8e2839ccd281823fb
+  metadata.gz: 99daea93402040c7584d5bf32550dc43d26a1189
+  data.tar.gz: ddbacdea54e3a9554fde47297f7a3215d1d2346d
 SHA512:
-  metadata.gz: 52805795500e3655a05eb660069a094348f767e44940a4182f1ec110b5a0c5793607dbbebcb1ce27d1ce5b6947fbf530327233f2604aa6eabb0dd8cd6a3bd170
-  data.tar.gz: 46432b91937362c31bc53aa4b62609d823bb7812e2f396c4ec38da585dac8d78bb360c699d99de6f799aeb996e8cf8422bc641221b405f38a3e2871b0ffcf816
+  metadata.gz: 966ae6c5066e393f93ba03225330fc7f669d24aff2075aea53c5e8601e7f9264ea7a8b7e504817e74d7ecd3b203fe4eba80124aca55eda187b599e1f23b0e9cb
+  data.tar.gz: ce6fe0b401e16bf63123692b09efc57adcdb881198e1460b196f48a90087a4f6d3ba3600b57be86bd495545dd7ceeb5e7b8551d3b322bbb81cefb7001fed0563

data/README.md

@@ -1,10 +1,10 @@
 Griddler::Sendgrid
 ==================
 
-This is an adapter that allows [Griddler](/thoughtbot/griddler) to be used with
+This is an adapter that allows [Griddler](https://github.com/thoughtbot/griddler) to be used with
 [SendGrid's Parse API].
 
-[SendGrid's Parse API]: http://sendgrid.com/docs/API%20Reference/Webhooks/parse.html
+[SendGrid's Parse API]: http://sendgrid.com/docs/API_Reference/Webhooks/parse.html
 
 Installation
 ------------
@@ -12,7 +12,6 @@ Installation
 Add this line to your application's Gemfile:
 
 ```ruby
-gem 'griddler'
 gem 'griddler-sendgrid'
 ```
 
@@ -20,16 +19,17 @@ Usage
 -----
 
 * SendGrid has done a [great
-  tutorial](http://blog.sendgrid.com/receiving-email-in-your-rails-app-with-griddler/)
+  tutorial](https://sendgrid.com/blog/receiving-email-in-your-rails-app-with-griddler/)
   on integrating Griddler with your application.
 * And of course, view our own blog post on the subject over at [Giant
   Robots](http://robots.thoughtbot.com/handle-incoming-email-with-griddler).
+* *Note:* Make sure to uncheck the "Spam Check" and "Send Raw" checkboxes on the [Parse Webhook settings page](http://sendgrid.com/developer/reply), otherwise the returned parsed email will have the body stripped out.
 
 More Information
 ----------------
 
 * [SendGrid](http://www.sendgrid.com)
-* [SendGrid Parse API](http://www.sendgrid.com/docs/API Reference/Webhooks/parse.html)
+* [SendGrid Parse API](http://www.sendgrid.com/docs/API_Reference/Webhooks/parse.html)
 
 Credits
 -------

data/griddler-sendgrid.gemspec

@@ -18,7 +18,7 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
 
   spec.add_development_dependency 'rake'
-  spec.add_development_dependency 'rspec'
+  spec.add_development_dependency 'rspec', '~> 3.5'
 
   spec.add_dependency 'griddler'
 end

data/lib/griddler/sendgrid/adapter.rb

@@ -12,8 +12,9 @@ module Griddler
 
       def normalize_params
         params.merge(
-          to: recipients(:to),
-          cc: recipients(:cc),
+          to: recipients(:to).map(&:format),
+          cc: recipients(:cc).map(&:format),
+          bcc: get_bcc,
           attachments: attachment_files,
         )
       end
@@ -23,17 +24,48 @@ module Griddler
       attr_reader :params
 
       def recipients(key)
-        ( params[key] || '' ).split(',')
+        Mail::AddressList.new(params[key] || '').addresses
       end
 
-      def attachment_files
-        params.delete('attachment-info')
-        attachment_count = params[:attachments].to_i
+      def get_bcc
+        if bcc = bcc_from_envelope
+          bcc - recipients(:to).map(&:address) - recipients(:cc).map(&:address)
+        else
+          []
+        end
+      end
+
+      def bcc_from_envelope
+        JSON.parse(params[:envelope])["to"] if params[:envelope].present?
+      end
 
+      def attachment_files
         attachment_count.times.map do |index|
-          params.delete("attachment#{index + 1}".to_sym)
+          extract_file_at(index)
+        end
+      end
+
+      def attachment_count
+        params[:attachments].to_i
+      end
+
+      def extract_file_at(index)
+        filename = attachment_filename(index)
+
+        params.delete("attachment#{index + 1}".to_sym).tap do |file|
+          if filename.present?
+            file.original_filename = filename
+          end
         end
       end
+
+      def attachment_filename(index)
+        attachment_info.fetch("attachment#{index + 1}", {})["filename"]
+      end
+
+      def attachment_info
+        @attachment_info ||= JSON.parse(params.delete("attachment-info") || "{}")
+      end
     end
   end
 end

data/lib/griddler/sendgrid/version.rb

@@ -1,5 +1,5 @@
 module Griddler
   module Sendgrid
-    VERSION = "0.0.1"
+    VERSION = "1.0.0"
   end
 end

data/spec/griddler/sendgrid/adapter_spec.rb

@@ -23,15 +23,15 @@ describe Griddler::Sendgrid::Adapter, '.normalize_params' do
       attachment2: upload_2,
      'attachment-info' => <<-eojson
         {
-          'attachment2': {
-            'filename': 'photo2.jpg',
-            'name': 'photo2.jpg',
-            'type': 'image/jpeg'
+          "attachment2": {
+            "filename": "photo2.jpg",
+            "name": "photo2.jpg",
+            "type": "image/jpeg"
           },
-          'attachment1': {
-            'filename': 'photo1.jpg',
-            'name': 'photo1.jpg',
-            'type': 'image/jpeg'
+          "attachment1": {
+            "filename": "photo1.jpg",
+            "name": "photo1.jpg",
+            "type": "image/jpeg"
           }
         }
       eojson
@@ -44,6 +44,33 @@ describe Griddler::Sendgrid::Adapter, '.normalize_params' do
     normalized_params.should_not have_key(:attachment_info)
   end
 
+  it "uses sendgrid attachment info for filename" do
+    params = default_params.merge(
+      attachments: "2",
+      attachment1: upload_1,
+      attachment2: upload_2,
+      "attachment-info" => <<-eojson
+        {
+          "attachment2": {
+            "filename": "sendgrid-filename2.jpg",
+            "name": "photo2.jpg",
+            "type": "image/jpeg"
+          },
+          "attachment1": {
+            "filename": "sendgrid-filename1.jpg",
+            "name": "photo1.jpg",
+            "type": "image/jpeg"
+          }
+        }
+      eojson
+    )
+
+    attachments = normalize_params(params)[:attachments]
+
+    attachments.first.original_filename.should eq "sendgrid-filename1.jpg"
+    attachments.second.original_filename.should eq "sendgrid-filename2.jpg"
+  end
+
   it 'has no attachments' do
     params = default_params.merge(attachments: '0')
 
@@ -51,10 +78,14 @@ describe Griddler::Sendgrid::Adapter, '.normalize_params' do
     normalized_params[:attachments].should be_empty
   end
 
-  it 'wraps to in an array' do
+  it 'splits to into an array' do
     normalized_params = normalize_params(default_params)
 
-    normalized_params[:to].should eq [default_params[:to]]
+    normalized_params[:to].should eq [
+      '"Mr Fugushima at Fugu, Inc" <hi@example.com>',
+      'Foo bar <foo@example.com>',
+      'no-name@example.com',
+    ]
   end
 
   it 'wraps cc in an array' do
@@ -70,12 +101,40 @@ describe Griddler::Sendgrid::Adapter, '.normalize_params' do
     normalized_params[:cc].should eq []
   end
 
+  it 'returns an array even if bcc is an empty string' do
+    params = default_params.merge(envelope: '')
+    normalized_params = normalize_params(params)
+
+    normalized_params[:bcc].should eq []
+  end
+
+  it 'wraps bcc in an array' do
+    normalized_params = normalize_params(default_params)
+
+    normalized_params[:bcc].should eq ["johny@example.com"]
+  end
+
+  it 'returns an array even if bcc is empty' do
+    params = default_params.merge(envelope: nil)
+    normalized_params = normalize_params(params)
+
+    normalized_params[:bcc].should eq []
+  end
+
+  it 'returns an empty array when the envelope to is the same as the base to' do
+    params = default_params.merge(envelope: "{\"to\":[\"hi@example.com\"]}")
+    normalized_params = normalize_params(params)
+
+    normalized_params[:bcc].should eq []
+  end
+
   def default_params
     {
       text: 'hi',
-      to: 'hi@example.com',
+      to: '"Mr Fugushima at Fugu, Inc" <hi@example.com>, Foo bar <foo@example.com>, <no-name@example.com>',
       cc: 'cc@example.com',
       from: 'there@example.com',
+      envelope: "{\"to\":[\"johny@example.com\"], \"from\": [\"there@example.com\"]}",
     }
   end
 end

data/spec/spec_helper.rb

@@ -3,8 +3,10 @@ require 'griddler/sendgrid'
 require 'action_dispatch'
 
 RSpec.configure do |config|
-  config.treat_symbols_as_metadata_keys_with_true_values = true
   config.run_all_when_everything_filtered = true
   config.order = 'random'
   config.include Griddler::Testing
+  config.expect_with(:rspec) do |c|
+    c.syntax = [:should, :expect]
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: griddler-sendgrid
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 1.0.0
 platform: ruby
 authors:
 - Caleb Thompson
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-06-16 00:00:00.000000000 Z
+date: 2017-01-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -28,16 +28,16 @@ dependencies:
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.5'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.5'
 - !ruby/object:Gem::Dependency
   name: griddler
   requirement: !ruby/object:Gem::Requirement
@@ -93,7 +93,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.5.2
 signing_key: 
 specification_version: 4
 summary: SendGrid adapter for Griddler