-
Notifications
You must be signed in to change notification settings - Fork 98
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
Support hapi tracing through the plugin API #379
Support hapi tracing through the plugin API #379
Conversation
PTAL |
src/plugins/plugin-hapi.js
Outdated
res.setHeader(constants.TRACE_CONTEXT_HEADER_NAME, context); | ||
|
||
// wrap end | ||
res.end = function(chunk, encoding) { |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
src/plugins/plugin-hapi.js
Outdated
var urlParse = require('url').parse; | ||
|
||
var constants = require('../constants.js'); | ||
var TraceLabels = require('../trace-labels.js'); |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
src/plugins/plugin-hapi.js
Outdated
}; | ||
api.runInRootSpan(options, function(transaction) { | ||
if (!transaction) { | ||
// TODO: Determine if this message is still needed |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
b487aa4
to
88d7ed3
Compare
Previously `constants.js`and `trace-labels.js` were also used.
e8dd755
to
4e3ae53
Compare
No description provided.