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.
Edited by Nick Thomas