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

[#9460] Fix path of reactor-netty server #9469

Merged

Conversation

jaehong-kim
Copy link
Contributor

@jaehong-kim jaehong-kim added this to the 2.5.0 milestone Dec 6, 2022
@jaehong-kim jaehong-kim self-assigned this Dec 6, 2022
@codecov
Copy link

codecov bot commented Dec 6, 2022

Codecov Report

Merging #9469 (a137f0f) into master (c2e1a40) will decrease coverage by 0.25%.
The diff coverage is 55.55%.

@@             Coverage Diff              @@
##             master    #9469      +/-   ##
============================================
- Coverage     39.16%   38.91%   -0.26%     
- Complexity    11744    11751       +7     
============================================
  Files          3417     3445      +28     
  Lines         91647    92296     +649     
  Branches      10166    10263      +97     
============================================
+ Hits          35894    35917      +23     
- Misses        52649    53273     +624     
- Partials       3104     3106       +2     
Impacted Files Coverage Δ
.../reactor/netty/interceptor/HttpRequestAdaptor.java 0.00% <0.00%> (ø)
...etty/interceptor/HttpServerParameterExtractor.java 0.00% <0.00%> (ø)
...int/plugin/reactor/netty/interceptor/UriUtils.java 68.18% <68.18%> (ø)
...n/server/bo/serializer/trace/v2/SpanEncoderV0.java 82.88% <0.00%> (-0.54%) ⬇️
...n/server/bo/serializer/trace/v2/SpanDecoderV0.java 81.27% <0.00%> (-0.46%) ⬇️
...r/HttpClientHandlerRequestWithBodyInterceptor.java 0.00% <0.00%> (ø)
...y/interceptor/HttpTcpClientConnectInterceptor.java 0.00% <0.00%> (ø)
...nt/plugin/reactor/netty/ReactorNettyConstants.java 0.00% <0.00%> (ø)
...tor/ChannelOperationsChannelMethodInterceptor.java 0.00% <0.00%> (ø)
... and 26 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@jaehong-kim jaehong-kim merged commit 5a68579 into pinpoint-apm:master Dec 6, 2022
@jaehong-kim jaehong-kim deleted the fix-reactor-netty-plugin-path branch December 6, 2022 07:10
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.

1 participant