This looks great, I just have a few questions before merging.

First up is here. Is there any reason we're using the global logger here, vs. self.server.logger?