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

Support Koa tracing through the plugin API #380

Merged
merged 5 commits into from
Feb 16, 2017

Conversation

DominicKramer
Copy link
Contributor

No description provided.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 10, 2017
@DominicKramer
Copy link
Contributor Author

PTAL

@DominicKramer DominicKramer force-pushed the plugin/support-koa branch 2 times, most recently from 82c2b80 to 379416a Compare February 15, 2017 22:42
Copy link
Contributor

@matthewloring matthewloring left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with a heads up/clarification.

}
//else {
// TODO: Determine if this is still needed
// agent.logger.warn('koa: Attempted to generate trace context for nullSpan');

This comment was marked as spam.

This comment was marked as spam.

@DominicKramer DominicKramer merged commit 0eb3896 into googleapis:master Feb 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants