-
Notifications
You must be signed in to change notification settings - Fork 32
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
Comments
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 |
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") |
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 filters = Seq(traceEndHookFilter) |
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 { |
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?The text was updated successfully, but these errors were encountered: