Should try/catch instead be around the inner code where we know we have the logger? Also, don't we have some kind of logging using/IDisposable mechanism that does that automatically without needing explicit exception handling?