search

balanced / balanced-api

Balanced API specification.
220 stars 72 forks source link

CRITICAL: 500 Internal Server Error intermittently when capturing a hold #600

Closed dcheckoway closed 10 years ago

dcheckoway commented 10 years ago

I mentioned this on IRC but I need to drop off...formalizing the bug report here. Hope this is the right place for it!

Create a hold. Works great. Capture the hold -- works most of the time, but sometimes (3x in the past week) we've seen a 500 Internal Server Error. For example:

https://dashboard.balancedpayments.com/#/marketplaces/MP3QAvY5JdkpDjmIKys2Ccub/logs/OHMb4aa3c52d6e711e3be3f02b12035401b

And when we retry the capture it says the hold is already captured, i.e.:

https://dashboard.balancedpayments.com/#/marketplaces/MP3QAvY5JdkpDjmIKys2Ccub/logs/OHMb74b48c0d6e711e39cf202b12035401b

This is a critical issue for us since it leaves our system in a completely wonky state. This just started happening about a week ago. I can give you log links for each of the failures if the ones above aren't conclusive/obvious enough.

If we (Lasso) are doing something wrong that is causing the 500 (as an API developer myself I would argue that the user should never be able to break the API, but I digress), please let us know...thanks!

matin commented 10 years ago

Ping @remear

remear commented 10 years ago

@dcheckoway It looks like some API timeouts caused these issues. You should be able to try these holds and captures again without issue. While I don't expect it to happen again, if you see this again, please let support know immediately.

A discussion regarding handling failed requests has been ongoing at https://github.com/balanced/balanced-api/issues/569 and may interest you.