Hacker News new | ask | show | jobs
by reader_mode 1778 days ago
I think this code is bad because it uses if/else (it should be if not abort) and it only tests/handles one error case and the log is not very valuable, would be much better to wrap the block in try catch and capture all exceptions (should be fairly obvious from exception that stripe.handleCardPayment doesn't exist)