search

mozilla / payments-service

This is a web service to support the Mozilla Payments web UI.
Other
5 stars 7 forks source link

Remove extra GET request during patch #91

Closed kumar303 closed 9 years ago

kumar303 commented 9 years ago

Was: https://github.com/mozilla/payments-service/pull/81

kumar303 commented 9 years ago

whoops, Andy might have another patch in the works for this already

andymckay commented 9 years ago

The other patch is there, but there's still a question of how payments-service is going to enforce security, should it?

andymckay commented 9 years ago

Said patch is here: https://github.com/mozilla/solitude/pull/501

kumar303 commented 9 years ago

I'm closing this PR because I do think we'll need the extra GET request. And, yes, I think the service should be responsible for making sure the logged in user owns the payment method that they are about to patch. Filed: https://github.com/mozilla/payments-service/issues/95