From ee2f21f09a7413fbf37e73aac8c177d8caacc3e3 Mon Sep 17 00:00:00 2001 From: Jorik Schellekens Date: Mon, 24 Jun 2019 16:42:39 +0100 Subject: [PATCH] Remove dependence on jaeger internals --- synapse/http/site.py | 4 ---- 1 file changed, 4 deletions(-) diff --git a/synapse/http/site.py b/synapse/http/site.py index 656299198f..4ab117aa4b 100644 --- a/synapse/http/site.py +++ b/synapse/http/site.py @@ -321,10 +321,6 @@ class SynapseRequest(Request): scope = opentracing.tracer.scope_manager.active if scope is not None: - # TODO: Remove this, it's just for debug and relies on implementation - # specific details of the jaeger_tracer - tags = {t.key: t.vStr for t in scope.span.tags} - assert(tags['request_id'] == self.get_request_id()) # finish the span if it's there. scope.span.set_tag("peer.address", authenticated_entity) scope.__exit__(None, None, None)