Use `withCallingHandlers()` in `log_failure()`

[hadley]

It's more appropriate to use withCallingHandlers() here because we want to add a side-effect, not change the return value. I doubt this will have much impact in practice, but it does make the tracebacks from errors in log_failure() a little simpler.