Skip to content

MLE-23593 Using forked copy of Spring's deprecated OkHttp support#792

Merged
rjrudin merged 1 commit intodevfrom
feature/okhttp
Sep 19, 2025
Merged

MLE-23593 Using forked copy of Spring's deprecated OkHttp support#792
rjrudin merged 1 commit intodevfrom
feature/okhttp

Conversation

@rjrudin
Copy link
Contributor

@rjrudin rjrudin commented Sep 19, 2025

Spring's support for OkHttp has long been deprecated and will be removed in Spring 7. This copies the 4 necessary classes from Spring into our codebase so that we can keep using OkHttp without any deprecation warnings.

@github-actions
Copy link

github-actions bot commented Sep 19, 2025

Copyright Validation Results
Total: 9 | Passed: 8 | Failed: 0 | Skipped: 1 | at: 2025-09-19 15:26:50 UTC | commit: 1e1b140

⏭️ Skipped (Excluded) Files

  • gradle.properties

✅ Valid Files

  • ml-app-deployer/src/main/java/com/marklogic/appdeployer/command/AbstractCommand.java
  • ml-app-deployer/src/main/java/com/marklogic/appdeployer/command/viewschemas/DeployViewSchemasCommand.java
  • ml-app-deployer/src/main/java/com/marklogic/mgmt/ManageClient.java
  • ml-app-deployer/src/main/java/com/marklogic/rest/util/RestTemplateUtil.java
  • ml-app-deployer/src/main/java/com/marklogic/rest/util/vendor/AbstractStreamingClientHttpRequest.java
  • ml-app-deployer/src/main/java/com/marklogic/rest/util/vendor/OkHttpClientHttpRequest.java
  • ml-app-deployer/src/main/java/com/marklogic/rest/util/vendor/OkHttpClientHttpRequestFactory.java
  • ml-app-deployer/src/main/java/com/marklogic/rest/util/vendor/OkHttpClientHttpResponse.java

✅ All files have valid copyright headers!

Spring's support for OkHttp has long been deprecated and will be removed in Spring 7. This copies the 4 necessary classes from Spring into our codebase so that we can keep using OkHttp without any deprecation warnings.
@rjrudin rjrudin merged commit b4b3ea0 into dev Sep 19, 2025
3 checks passed
@rjrudin rjrudin deleted the feature/okhttp branch September 19, 2025 16:08
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.

1 participant