Fix null span bug in Zuul filter
If a request is not being sampled, the ClientReceivedEvent should not be fired. Fixes gh-47
This commit is contained in:
@@ -29,8 +29,8 @@ import com.netflix.zuul.ZuulFilter;
|
||||
* @author Dave Syer
|
||||
*
|
||||
*/
|
||||
public class TracePostZuulFilter extends ZuulFilter implements
|
||||
ApplicationEventPublisherAware {
|
||||
public class TracePostZuulFilter extends ZuulFilter
|
||||
implements ApplicationEventPublisherAware {
|
||||
|
||||
private ApplicationEventPublisher publisher;
|
||||
|
||||
@@ -41,7 +41,7 @@ ApplicationEventPublisherAware {
|
||||
|
||||
@Override
|
||||
public boolean shouldFilter() {
|
||||
return true;
|
||||
return getCurrentSpan() != null;
|
||||
}
|
||||
|
||||
@Override
|
||||
|
||||
Reference in New Issue
Block a user