From 96e4c2b1930bf5d1a3c2fbd2b6a7c4162df281d9 Mon Sep 17 00:00:00 2001 From: Lev Khomich Date: Sun, 4 Oct 2015 03:45:51 +0500 Subject: [PATCH] do not record message contents in ActorTracing instrumentation --- .../com/github/levkhomich/akka/tracing/ActorTracing.scala | 1 - .../com/github/levkhomich/akka/tracing/ActorTracingSpec.scala | 3 ++- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/core/src/main/scala/com/github/levkhomich/akka/tracing/ActorTracing.scala b/core/src/main/scala/com/github/levkhomich/akka/tracing/ActorTracing.scala index 99c468d..9211a5a 100644 --- a/core/src/main/scala/com/github/levkhomich/akka/tracing/ActorTracing.scala +++ b/core/src/main/scala/com/github/levkhomich/akka/tracing/ActorTracing.scala @@ -49,7 +49,6 @@ trait ActorTracing extends AroundReceiveOverrideHack { self: Actor => msg match { case ts: BaseTracingSupport if receive.isDefinedAt(msg) => trace.start(ts, serviceName) - trace.record(ts, "request: " + msg.toString) case _ => } } diff --git a/core/src/test/scala/com/github/levkhomich/akka/tracing/ActorTracingSpec.scala b/core/src/test/scala/com/github/levkhomich/akka/tracing/ActorTracingSpec.scala index 908a790..87ae51e 100644 --- a/core/src/test/scala/com/github/levkhomich/akka/tracing/ActorTracingSpec.scala +++ b/core/src/test/scala/com/github/levkhomich/akka/tracing/ActorTracingSpec.scala @@ -40,7 +40,8 @@ class ActorTracingSpec extends Specification with TracingTestCommons with Tracin trace.finish(message) val span = receiveSpan() - checkAnnotation(span, "request: " + message) + span.get_annotations.size mustEqual 2 + span.get_binary_annotations mustEqual null } }