Seems fine...until you realise that this is the condition to exit early, it's still logging 90% of the time which is almost certainly not the desired behavior.
Not to mention _ok is a terrible name for that variable, and it's only ever being used as a negation. Why isn't it called _error or something?
Not to mention _ok is a terrible name for that variable, and it's only ever being used as a negation. Why isn't it called _error or something?