Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

activator-play-tracing with Play 2.5 #67

Open
austek opened this issue Apr 1, 2016 · 4 comments
Open

activator-play-tracing with Play 2.5 #67

austek opened this issue Apr 1, 2016 · 4 comments
Labels

Comments

@austek
Copy link

austek commented Apr 1, 2016

Is it possible to either list what changes to make to activator-play-tracing or update the project to make it work with Play 2.5 as they have made some breaking changes and I couldn't get tracing to work?

@austek
Copy link
Author

austek commented Apr 10, 2016

I have started migration to work with play 2.5, but currently all doesn't work. Connection to zipkin server is successful but trace are not submitted

#71
levkhomich/activator-play-tracing#1

@monkeygroover
Copy link

its because of this in GlobalSettings, which TracingSettings overrides.

i'll see if i can fix it.

@deprecated("onRequestCompletion is no longer invoked by Play. The same functionality can be achieved by adding a filter that attaches a onDoneEnumerating callback onto the returned Result Enumerator.", "2.4.0")
def onRequestCompletion(request: RequestHeader) {
}

@monkeygroover
Copy link

monkeygroover@f35e746

quick test to see if a filter will work. (note I only tried 2.4 not 2.5 yet)

add, this in the root of the project.

class Filters extends HttpFilters {

import play.api.libs.concurrent.Execution.Implicits._
val traceEndHookFilter = new TracingFilter()

val filters = Seq(traceEndHookFilter)
}

@tsteff
Copy link

tsteff commented Jan 18, 2017

I've tried this approach myself but when it tries to submit the span I receive the following frame size error:

WARN [application-akka.actor.default-dispatcher-2] [Slf4jLogger.scala:76] - Thrift transport error: Frame size (1213486160) larger than max length (16384000)!. Failed to send 1 spans. - [sourceThread=application-akka.actor.default-dispatcher-5, akkaSource=akka://application/user/spanHolder/spanSubmitter, sourceActorSystem=application, akkaTimestamp=12:45:52.831UTC] - [] -

class Filters @Inject()(tracingFilter: TracingFilter) extends HttpFilters {
val filters = Seq(tracingFilter)
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants