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

feat(traceparent): setting parent span from server #477

Merged
merged 13 commits into from
Nov 8, 2019
Merged
11 changes: 11 additions & 0 deletions examples/tracer-web/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,17 @@
<title>Web Tracer Example</title>
<base href="/">

<!--
https://www.w3.org/TR/trace-context/
Set the `traceparent` in the server's HTML template code. It should be
dynamically generated server side to have the server's request trace Id,
a parent span Id that was set on the server's request span, and the trace
flags to indicate the server's sampling decision
(01 = sampled, 00 = notsampled).
'{version}-{traceId}-{spanId}-{sampleDecision}'
-->
<meta name="traceparent" content="00-ab42124a3c573678d4d8b21ba52df3bf-d21f7bc17caa5aba-01">

<meta name="viewport" content="width=device-width, initial-scale=1">
</head>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,17 @@ const VALID_SPANID_REGEX = /^[0-9a-f]{16}$/i;
const INVALID_ID_REGEX = /^0+$/i;
const VERSION = '00';

function parse(traceParent: string): SpanContext | null {
/**
* Parses information from the [traceparent] span tag and converts it into {@link SpanContext}
* @param traceParent - A meta property that comes from server.
* It should be dynamically generated server side to have the server's request trace Id,
* a parent span Id that was set on the server's request span,
* and the trace flags to indicate the server's sampling decision
* (01 = sampled, 00 = not sampled).
* for example: '{version}-{traceId}-{spanId}-{sampleDecision}'
* For more information see {@link https://www.w3.org/TR/trace-context/}
*/
export function parseTraceParent(traceParent: string): SpanContext | null {
const match = traceParent.match(VALID_TRACE_PARENT_REGEX);
if (!match) return null;
const parts = traceParent.split('-');
Expand Down Expand Up @@ -87,7 +97,7 @@ export class HttpTraceContext implements HttpTextFormat {
const traceParent = Array.isArray(traceParentHeader)
? traceParentHeader[0]
: traceParentHeader;
const spanContext = parse(traceParent);
const spanContext = parseTraceParent(traceParent);
if (!spanContext) return null;

spanContext.isRemote = true;
Expand Down
29 changes: 29 additions & 0 deletions packages/opentelemetry-plugin-document-load/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,34 @@ const webTracer = new WebTracer({
webTracer.addSpanProcessor(new SimpleSpanProcessor(new ConsoleSpanExporter()));
```

## Optional: Send a trace parent from your server
This plugin supports connecting the server side spans for the initial HTML load with the client side span for the load from the browser's timing API. This works by having the server send its parent trace context (trace ID, span ID and trace sampling decision) to the client.

Because the browser does not send a trace context header for the initial page navigation, the server needs to fake a trace context header in a middleware and then send that trace context header back to the client as a meta tag *traceparent* . The *traceparent* meta tag should be in the [trace context W3C draft format][trace-context-url] . For example:

```html
...
<head>
<!--
https://www.w3.org/TR/trace-context/
Set the `traceparent` in the server's HTML template code. It should be
dynamically generated server side to have the server's request trace Id,
a parent span Id that was set on the server's request span, and the trace
flags to indicate the server's sampling decision
(01 = sampled, 00 = notsampled).
'{version}-{traceId}-{spanId}-{sampleDecision}'
-->
<meta name="traceparent" content="00-ab42124a3c573678d4d8b21ba52df3bf-d21f7bc17caa5aba-01">
</head>
<body>
...
<script>
// and then initialise the WebTracer
// var webTracer = new WebTracer({ .......
</script>
</body>
```

## Useful links
- For more information on OpenTelemetry, visit: <https://opentelemetry.io/>
- For more about OpenTelemetry JavaScript: <https://github.com/open-telemetry/opentelemetry-js>
Expand All @@ -48,3 +76,4 @@ Apache 2.0 - See [LICENSE][license-url] for more information.
[devDependencies-url]: https://david-dm.org/open-telemetry/opentelemetry-js?path=packages%2Fopentelemetry-plugin-document-load&type=dev
[npm-url]: https://www.npmjs.com/package/@opentelemetry/plugin-document-load
[npm-img]: https://badge.fury.io/js/%40opentelemetry%2Fplugin-document-load.svg
[trace-context-url]: https://www.w3.org/TR/trace-context
16 changes: 14 additions & 2 deletions packages/opentelemetry-plugin-document-load/src/documentLoad.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,12 @@
* limitations under the License.
*/

import { BasePlugin, otperformance } from '@opentelemetry/core';
import {
BasePlugin,
otperformance,
parseTraceParent,
TRACE_PARENT_HEADER,
} from '@opentelemetry/core';
import { PluginConfig, Span, SpanOptions } from '@opentelemetry/types';
import { AttributeNames } from './enums/AttributeNames';
import { PerformanceTimingNames as PTN } from './enums/PerformanceTimingNames';
Expand Down Expand Up @@ -106,12 +111,19 @@ export class DocumentLoad extends BasePlugin<unknown> {
* Collects information about performance and creates appropriate spans
*/
private _collectPerformance() {
const metaElement = [...document.getElementsByTagName('meta')].find(
e => e.getAttribute('name') === TRACE_PARENT_HEADER
);
const serverContext =
parseTraceParent((metaElement && metaElement.content) || '') || undefined;

const entries = this._getEntries();

const rootSpan = this._startSpan(
AttributeNames.DOCUMENT_LOAD,
PTN.FETCH_START,
entries
entries,
{ parent: serverContext }
);
if (!rootSpan) {
return;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
import * as assert from 'assert';
import * as sinon from 'sinon';

import { ConsoleLogger } from '@opentelemetry/core';
import { ConsoleLogger, TRACE_PARENT_HEADER } from '@opentelemetry/core';
import {
BasicTracer,
ReadableSpan,
Expand Down Expand Up @@ -306,6 +306,53 @@ describe('DocumentLoad Plugin', () => {
done();
});
});

describe('AND window has information about server root span', () => {
let spyGetElementsByTagName: any;
beforeEach(() => {
const element = {
content: '00-ab42124a3c573678d4d8b21ba52df3bf-d21f7bc17caa5aba-01',
getAttribute: (value: string) => {
if (value === 'name') {
return TRACE_PARENT_HEADER;
}
return undefined;
},
};

spyGetElementsByTagName = sinon.stub(
window.document,
'getElementsByTagName'
);
spyGetElementsByTagName.withArgs('meta').returns([element]);
});
afterEach(() => {
spyGetElementsByTagName.restore();
});

it('should create a root span with server context traceId', done => {
const spyOnEnd = sinon.spy(dummyExporter, 'export');
plugin.enable(moduleExports, tracer, logger, config);
setTimeout(() => {
const rootSpan = spyOnEnd.args[0][0][0] as ReadableSpan;
const fetchSpan = spyOnEnd.args[1][0][0] as ReadableSpan;
assert.strictEqual(rootSpan.name, 'documentFetch');
assert.strictEqual(fetchSpan.name, 'documentLoad');

assert.strictEqual(
rootSpan.spanContext.traceId,
'ab42124a3c573678d4d8b21ba52df3bf'
);
assert.strictEqual(
fetchSpan.spanContext.traceId,
'ab42124a3c573678d4d8b21ba52df3bf'
);

assert.strictEqual(spyOnEnd.callCount, 2);
done();
}, 1);
});
});
});

describe('when resource entries are available', () => {
Expand Down