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
Add config allowing classes to be excluded from transformation #807
Conversation
Add the following system property or corresponding environment variable: ``` dd.trace.classes.exclude=some.packagename.*,some.classname.MyClass$NestedClass ```
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a couple of questions but I this this is good to go in either case
addDecorator(decorator); | ||
} | ||
|
||
registerClassLoader(ClassLoader.getSystemClassLoader()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this line related to this PR or a left over?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was a very minor change intended to improve comprehension of log statement order. I'll try to reduce unrelated changes in PRs going forward.
@@ -42,6 +38,7 @@ public void handle(final Context ctx) { | |||
DECORATE.afterStart(ratpackSpan); | |||
DECORATE.onConnection(ratpackSpan, request); | |||
DECORATE.onRequest(ratpackSpan, request); | |||
ctx.getExecution().add(ratpackSpan); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this LOC is related to this PR right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, I slipped in a few other misc changes.
public FluxAndMonoInstrumentation() { | ||
super("reactor-core"); | ||
} | ||
|
||
@Override | ||
public String[] helperClassNames() { | ||
return new String[] { | ||
PACKAGE + ".ReactorCoreAdviceUtils", PACKAGE + ".ReactorCoreAdviceUtils$TracingSubscriber" | ||
packageName + ".ReactorCoreAdviceUtils", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think these LOCs are related to this PR?
Add the following system property or corresponding environment variable: