Skip to content

Commit

Permalink
Merge pull request pinpoint-apm#2078 from Xylus/bugfix/issue-2077
Browse files Browse the repository at this point in the history
Fixes StandardHostValve.invoke() not being transformed when excludeurl is not set
  • Loading branch information
emeroad authored Sep 8, 2016
2 parents 0f94b21 + 6f90e3c commit 4f7a442
Showing 1 changed file with 6 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -14,24 +14,24 @@
*/
package com.navercorp.pinpoint.plugin.tomcat;

import com.navercorp.pinpoint.bootstrap.config.ExcludePathFilter;
import com.navercorp.pinpoint.bootstrap.config.Filter;
import com.navercorp.pinpoint.bootstrap.config.ProfilerConfig;
import com.navercorp.pinpoint.bootstrap.config.SkipFilter;

/**
* @author Jongho Moon
*
*/
public class TomcatConfiguration {
private final boolean tomcatHidePinpointHeader;
private Filter<String> tomcatExcludeUrlFilter;
private final Filter<String> tomcatExcludeUrlFilter;

public TomcatConfiguration(ProfilerConfig config) {
this.tomcatHidePinpointHeader = config.readBoolean("profiler.tomcat.hidepinpointheader", true);
final String tomcatExcludeURL = config.readString("profiler.tomcat.excludeurl", "");

if (!tomcatExcludeURL.isEmpty()) {
this.tomcatExcludeUrlFilter = new ExcludePathFilter(tomcatExcludeURL);
if (config.getTomcatExcludeProfileMethodFilter() == null) {
this.tomcatExcludeUrlFilter = new SkipFilter<String>();
} else {
this.tomcatExcludeUrlFilter = config.getTomcatExcludeUrlFilter();
}
}

Expand Down

0 comments on commit 4f7a442

Please # to comment.