Merge pull request #86 from Everapps/callback-webhook

Stop passing `ActionController::Parameters` to `Stripe::Event.construct_from`
tansengming · Aug 29, 2017 · b1353f7 · b1353f7
2 parents 60adf3b + 0d78ff5
commit b1353f7
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/app/models/stripe/event_dispatch.rb b/app/models/stripe/event_dispatch.rb
@@ -11,7 +11,7 @@ def dispatch_stripe_event(params)
     def retrieve_stripe_event(params)
       id = params['id']
       if id == 'evt_00000000000000' #this is a webhook test
-        yield Stripe::Event.construct_from(params)
+        yield Stripe::Event.construct_from(params.to_unsafe_h)
       else
         yield Stripe::Event.retrieve(id)
       end

diff --git a/test/callbacks_spec.rb b/test/callbacks_spec.rb
@@ -53,6 +53,12 @@
         ->{ subject }.must_raise RuntimeError
       end
     end
+
+    describe 'when run from a Stripe webhook test' do
+      before { event['id'] = 'evt_00000000000000' }
+
+      it { subject } # must_not raise error
+    end
   end
 
   describe 'defined without a bang and raising an exception' do