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

[#10832] Backport: Support getStatusCode() for compatibility with S… #10833

Merged

Conversation

intr3p1d
Copy link
Contributor

@intr3p1d intr3p1d commented Apr 2, 2024

…pring WebFlux 6.1 & Boot 3.x.

[#NotAssigned] Removed getRawStatusCode() method for removal in Spring Webflux 6.1

[#10736] Separate dummy java class not to disturb spring-web dependency

[#10736] Remove dummy classes after install the plugin

[#10736] Use maven-clean-plugin instead of exec

[#10736] Split spring-stub into separate modules

…ility with Spring WebFlux 6.1 & Boot 3.x.

[#NotAssigned] Removed getRawStatusCode() method for removal in Spring Webflux 6.1

[pinpoint-apm#10736] Separate dummy java class not to disturb spring-web dependency

[pinpoint-apm#10736] Remove dummy classes after install the plugin

[pinpoint-apm#10736] Use maven-clean-plugin instead of exec

[pinpoint-apm#10736] Split spring-stub into separate modules
@intr3p1d intr3p1d added this to the 2.5.4 milestone Apr 2, 2024
@intr3p1d intr3p1d self-assigned this Apr 2, 2024
Copy link

codecov bot commented Apr 2, 2024

Codecov Report

Attention: Patch coverage is 0% with 45 lines in your changes are missing coverage. Please review.

Project coverage is 38.76%. Comparing base (d11d9e6) to head (91f2cdc).

Files Patch % Lines
...rp/pinpoint/plugin/resttemplate/SpringVersion.java 0.00% 12 Missing ⚠️
...te/interceptor/util/Spring5HttpStatusProvider.java 0.00% 7 Missing ⚠️
...te/interceptor/util/Spring6HttpStatusProvider.java 0.00% 7 Missing ⚠️
...te/interceptor/util/HttpStatusProviderFactory.java 0.00% 6 Missing ⚠️
...npoint/plugin/resttemplate/RestTemplatePlugin.java 0.00% 4 Missing ⚠️
...plate/interceptor/ListenableFutureInterceptor.java 0.00% 3 Missing ⚠️
...ate/interceptor/ClientHttpResponseInterceptor.java 0.00% 2 Missing ⚠️
...sttemplate/interceptor/HttpRequestInterceptor.java 0.00% 2 Missing ⚠️
...nterceptor/util/UnsupportedHttpStatusProvider.java 0.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##              2.5.x   #10833      +/-   ##
============================================
- Coverage     38.78%   38.76%   -0.02%     
+ Complexity    11926    11922       -4     
============================================
  Files          3538     3543       +5     
  Lines         94264    94301      +37     
  Branches      10504    10505       +1     
============================================
- Hits          36561    36559       -2     
- Misses        54530    54566      +36     
- Partials       3173     3176       +3     

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

@intr3p1d intr3p1d merged commit 1c0d070 into pinpoint-apm:2.5.x Apr 2, 2024
1 of 3 checks passed
@JooHyukKim
Copy link

May I ask the release date of 2.5.4 version? @intr3p1d

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.

2 participants