Don't finish the opentracing span early

Calling finished() in ContextWithCorrelationID breaks opentracing, since it expects us to call it just before exiting, and this defer runs on function completion.

All existing users of ContextWithCorrelationID already defer finished() themselves, so this call is entirely surplus to requirements.