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

[#10637] Added trace method filter #10639

Merged
merged 1 commit into from
Jan 23, 2024

Conversation

smilu97
Copy link
Contributor

@smilu97 smilu97 commented Jan 22, 2024

resolves #10637

@smilu97 smilu97 self-assigned this Jan 22, 2024
@smilu97 smilu97 changed the title [#noissue] Added pre exclude method filter [#10637] Added pre exclude method filter Jan 22, 2024
Copy link

codecov bot commented Jan 22, 2024

Codecov Report

Attention: 50 lines in your changes are missing coverage. Please review.

Comparison is base (c291612) 37.88% compared to head (81c8ab4) 37.85%.
Report is 3 commits behind head on master.

Files Patch % Lines
...otstrap/plugin/request/ServletRequestListener.java 0.00% 6 Missing ⚠️
...n/akka/http/interceptor/DirectivesInterceptor.java 0.00% 6 Missing ⚠️
...ptor/ServerConnectionHandleRequestInterceptor.java 0.00% 6 Missing ⚠️
.../plugin/request/ServletRequestListenerBuilder.java 0.00% 5 Missing ⚠️
...inpoint/plugin/weblogic/WeblogicConfiguration.java 0.00% 4 Missing ⚠️
...navercorp/pinpoint/plugin/tomcat/TomcatConfig.java 0.00% 3 Missing ⚠️
...corp/pinpoint/plugin/akka/http/AkkaHttpConfig.java 0.00% 2 Missing ⚠️
...plugin/reactor/netty/ReactorNettyPluginConfig.java 0.00% 2 Missing ⚠️
...ercorp/pinpoint/bootstrap/config/ServerConfig.java 66.66% 1 Missing ⚠️
...ap/plugin/request/ServerRequestWrapperAdaptor.java 0.00% 1 Missing ⚠️
... and 14 more
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #10639      +/-   ##
============================================
- Coverage     37.88%   37.85%   -0.04%     
+ Complexity    11887    11870      -17     
============================================
  Files          3795     3791       -4     
  Lines         94145    94091      -54     
  Branches      10228    10215      -13     
============================================
- Hits          35667    35617      -50     
  Misses        55532    55532              
+ Partials       2946     2942       -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@smilu97 smilu97 force-pushed the 10637_method_filter branch 2 times, most recently from 2cd09e3 to 31e66a2 Compare January 22, 2024 12:03
@smilu97 smilu97 changed the title [#10637] Added pre exclude method filter [#10637] Added trace method filter Jan 23, 2024
@smilu97 smilu97 merged commit ce264f6 into pinpoint-apm:master Jan 23, 2024
1 of 3 checks passed
@smilu97 smilu97 deleted the 10637_method_filter branch January 23, 2024 01:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature: Ignore http servlet requests which have designated methods
1 participant