Uh oh!
There was an error while loading. Please reload this page.
Conversation
Uh oh!
There was an error while loading. Please reload this page.
kamilogorek
commented
Aug 13, 2019
cc @mitsuhiko for last commit |
| const span = top.scope.getSpan(); | ||
| if (span) { | ||
| return span.newSpan(spanContext); |
There was a problem hiding this comment.
Let's call this childSpan instead of newSpan
Uh oh!
There was an error while loading. Please reload this page.
ajacques
commented
Aug 28, 2019
What's the long-term plan around APM in Sentry? Is there overlap with or a plan to integrate with OpenCensus/OpenTelemetry? OpenCensus just released a prototypeJavaScript implementation and I'd love to avoid having to integrate 2-3 different tracing systems. |
kamilogorek
commented
Sep 10, 2019
@ajacques we're still in early PoC phase so there's not much I can say for certain, but we do have OC/OT in mind while working on it :) |
HazAT
commented
Oct 4, 2019
TODO: |
Bessonov
commented
Oct 24, 2019
Hi guys, I found the blog post about Sentry APM. As I understand, Sentry wants to support not only error logging, but also tracing, right? Is there any plan to support "normal" Logging for better analysability of errors or do we still need tools like ELK? |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
| return TransactionActivity._enabled; | ||
| } | ||
| // This happens only in test cases where the integration isn't initalized properly | ||
| if (!TransactionActivity.options || isNaN(TransactionActivity.options.tracesSampleRate)) { |
There was a problem hiding this comment.
Remember that isNaN('')isNaN([])isNaN(null)isNaN(true)isNaN(false) will also report false. You can wrap the input in parseInt to make it better, or use:
typeof TransactionActivity.options.tracesSampleRate !== 'number'
| // Reason being at the time we start the inital transaction we do not have a client bound on the hub yet | ||
| // therefore configureScope wouldn't be executed and we would miss setting the transaction | ||
| // tslint:disable-next-line: no-unsafe-any | ||
| (hub as any).getScope().setSpan(span); |
dashed
commented
Dec 1, 2019
@HazAT@kamilogorek Can the span status also be part of the trace/span context? |
dashed
commented
Dec 2, 2019
Damn. So fast. |
This PR provides the new
@sentry/apmpackage with a newTracingintegration.The
Tracingintegration creates Transactions for pageloads as well as navigation changes. Additionally, it will add thesentry-traceheader so if used together with an apm enabled server SDK of Sentry (Node, Python) you will get the full trace.We also hook into global XHR & fetch calls and create spans out of it that will be attached to the transaction.