can we just make the logger required? I see no good reason why this should be optional.