-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Open
Labels
ClientThis issue points to a problem in the data-plane of the library.This issue points to a problem in the data-plane of the library.azure-springAll azure-spring related issuesAll azure-spring related issuesazure-spring-jcafeature-requestThis issue requires a new behavior in the product in order be resolved.This issue requires a new behavior in the product in order be resolved.
Milestone
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
HttpClientevery time an HTTP request needs to be made and instead use a longer-lived instance wherever possible. - Stop returning
nullin methods such asget()orpost()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.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
ClientThis issue points to a problem in the data-plane of the library.This issue points to a problem in the data-plane of the library.azure-springAll azure-spring related issuesAll azure-spring related issuesazure-spring-jcafeature-requestThis issue requires a new behavior in the product in order be resolved.This issue requires a new behavior in the product in order be resolved.
Type
Projects
Status
Todo