Azure / Azure/azure-sdk-for-java

[FEATURE REQ][Key Vault JCA] QoL changes for `HttpUtil`.

Open
#40,956 2 comments 0 reactions 2 assignees Claimed by @vcolin7 View on GitHub
azure-spring azure-spring-jca Client feature-request
Dominant language
Java
Stars
2.6k
Forks
2.2k
Avg merge
2d 8h
Merged PRs (30d)
178

Description

Just as outlined by #23547, the Key Vault JCA library is not up to par with other similar data plane libraries in our Azure SDK for Java. As part of the effort to align the library with the rest, we also need to add QoL changes to the `HttpUtil` class such as:
- Avoid creating a new `HttpClient` every time an HTTP request needs to be made and instead use a longer-lived instance wherever possible.
- Stop returning `null` in methods such as `get()` or `post()` when there's an error in sending a request/ receiving a response and instead bubble up a proper exception.
- Add logic to retry on transient errors, much like what is done in Azure Core's `RetryPolicy`.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.