I'd expect 'on_error' to take a function as a callback, not a string. But yes, a better named function could be added