Skip to content

Conversation

@kaajaln2
Copy link
Contributor

@kaajaln2 kaajaln2 commented Dec 9, 2025

Spring Framework 7 will remove the deprecated method getRawStatusCode() from the ClientHttpResponse interface.
Update the usage of getRawStatusCode() in the Apache Geode project’s HttpRequester.java to use the Spring-recommended method getStatusCode().value() from the same interface.
All unit tests ran successfully.

For all changes, please confirm:

  • Is there a JIRA ticket associated with this PR? Is it referenced in the commit message? yes
  • Has your PR been rebased against the latest commit within the target branch (typically develop)? yes
  • Is your initial contribution a single, squashed commit? yes
  • Does gradlew build run cleanly? yes
  • Have you written or updated unit tests to verify your changes? existing tests run clean
  • If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

@JinwooHwang JinwooHwang self-requested a review December 9, 2025 17:32
@JinwooHwang
Copy link
Contributor

Initiated checks.

@kaajaln2 kaajaln2 marked this pull request as ready for review December 9, 2025 22:47
Copy link
Contributor

@JinwooHwang JinwooHwang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This replaces getRawStatusCode() with response.getStatusCode().value() and updates the 401/403 checks accordingly. Behavior remains the same while avoiding deprecated API usage. Tested and looks good; approving the change.

@JinwooHwang JinwooHwang merged commit 64fe780 into apache:develop Dec 10, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants