Maybe I'm going crazy here (likely!), but should there be a ...
if (err) return callback(err);
... on the following line?
I have to imagine that we'd be seeing some sort of issue by now if we really needed this, but it seemed strange so I figured it was worth a quick sanity check.