Feature: add missing callbacks

[4nd2in]

• Add the following callbacks:
  • checkout.session.async_payment_failed
  • checkout.session.async_payment_succeeded
  • customer.subscription.paused
  • customer.subscription.pending_update_applied
  • customer.subscription.pending_update_expired
  • customer.subscription.resumed
  • invoice.deleted
  • invoice.finalization_failed
  • invoice.overdue
  • invoice.will_be_due
• Fix request handling in rack versions >= 3.1 where Request#[] is deprecated (undefined method `[]' for an instance of ActionDispatch::Request #230)

[light-flight]

Great job! Waiting for your fix to be merged

[tansengming]

thanks for the PR! Once you remove the change to def retrieve_stripe_event(request) this looks good to go.

[tansengming]

Hi @4nd2in can you split this change to it's own PR please? We might have to figure out a way to deal with this for rack versions that still use the old way.

[light-flight]

Hi, let me help.
stripe-rails supports rails '>= 5.1'
https://github.com/tansengming/stripe-rails/blob/master/stripe-rails.gemspec#L17

So rails 5.1.7 requires rack '~> 2.0'
https://github.com/rails/rails/blob/5-1-stable/actionpack/actionpack.gemspec#L29

I installed rack 2.0.1 locally. Created simple app:

# config.ru

app = Proc.new do |env|
  request = Rack::Request.new(env)
  [200, {}, [request.params['id']]]
end

run app

Run puma
http://localhost:9292/?id=12
gives me correct result.

Note that using a symbol for id does not work in this example!

So older versions of rack support request.params['id']. We do not need to do anything.

[light-flight]

Created separate PR
#232

[tansengming]

callbacks verified on https://docs.stripe.com/api/events/types

[tansengming]

looks good thanks for the help @4nd2in !