Always using default httpPipeline

Author: jpalvarezl

sdk/ai/azure-ai-agents/src/main/java/com/azure/ai/agents/AgentsClientBuilder.java

@@ -387,12 +387,10 @@ private OpenAIOkHttpClient.Builder getOpenAIClientBuilder() {
             .credential(
                 BearerTokenCredential.create(TokenUtils.getBearerTokenSupplier(this.tokenCredential, DEFAULT_SCOPES)));
         builder.baseUrl(this.endpoint + (this.endpoint.endsWith("/") ? "openai" : "/openai"));
-        builder.replaceHeaders("User-Agent", getUserAgent());
         if (this.serviceVersion != null) {
             builder.azureServiceVersion(AzureOpenAIServiceVersion.fromString(this.serviceVersion.getVersion()));
             builder.azureUrlPathMode(AzureUrlPathMode.UNIFIED);
         }
-        builder.timeout(Duration.ofSeconds(30));
         return builder;
     }
 
@@ -401,37 +399,15 @@ private OpenAIOkHttpClientAsync.Builder getOpenAIAsyncClientBuilder() {
             .credential(
                 BearerTokenCredential.create(TokenUtils.getBearerTokenSupplier(this.tokenCredential, DEFAULT_SCOPES)));
         builder.baseUrl(this.endpoint + (this.endpoint.endsWith("/") ? "openai" : "/openai"));
-        builder.replaceHeaders("User-Agent", getUserAgent());
         if (this.serviceVersion != null) {
             builder.azureServiceVersion(AzureOpenAIServiceVersion.fromString(this.serviceVersion.getVersion()));
             builder.azureUrlPath(AzureUrlPathMode.UNIFIED);
         }
-        builder.timeout(Duration.ofSeconds(30));
         return builder;
     }
 
-    private String getUserAgent() {
-        HttpLogOptions localHttpLogOptions = this.httpLogOptions == null ? new HttpLogOptions() : this.httpLogOptions;
-        ClientOptions localClientOptions = this.clientOptions == null ? new ClientOptions() : this.clientOptions;
-        String sdkName = PROPERTIES.getOrDefault(SDK_NAME, "UnknownName");
-        String sdkVersion = PROPERTIES.getOrDefault(SDK_VERSION, "UnknownVersion");
-        String applicationId = CoreUtils.getApplicationId(localClientOptions, localHttpLogOptions);
-        return UserAgentUtil.toUserAgentString(applicationId, sdkName, sdkVersion, configuration);
-    }
-
     private HttpClient getOpenAIHttpClient() {
-        if (this.httpClient == null) {
-            return null;
-        }
-        return new PolicyDecoratingHttpClient(this.httpClient, getOrderedCustomPolicies());
-    }
-
-    private List<HttpPipelinePolicy> getOrderedCustomPolicies() {
-        return this.pipelinePolicies.stream()
-            .sorted(Comparator.comparing(policy -> HttpPipelinePosition.PER_CALL == policy.getPipelinePosition()
-                ? HttpPipelinePosition.PER_CALL
-                : HttpPipelinePosition.PER_CALL))
-            .collect(Collectors.toList());
+        return new PolicyDecoratingHttpClient(createHttpPipeline());
     }
 
     private static final ClientLogger LOGGER = new ClientLogger(AgentsClientBuilder.class);

sdk/ai/azure-ai-agents/src/main/java/com/azure/ai/agents/implementation/http/HttpClientHelper.java

@@ -154,7 +154,7 @@ private static BinaryData toBinaryData(HttpRequestBody requestBody) {
             if (requestBody == null) {
                 return null;
             }
-
+            
             try (ByteArrayOutputStream outputStream = new ByteArrayOutputStream()) {
                 requestBody.writeTo(outputStream);
                 return BinaryData.fromBytes(outputStream.toByteArray());

sdk/ai/azure-ai-agents/src/main/java/com/azure/ai/agents/implementation/http/PolicyDecoratingHttpClient.java

@@ -22,7 +22,7 @@
  */
 public final class PolicyDecoratingHttpClient implements HttpClient {
 
-    private final HttpClient delegate;
+//    private final HttpClient delegate;
     private final HttpPipeline pipeline;
 
     /**
@@ -31,17 +31,21 @@ public final class PolicyDecoratingHttpClient implements HttpClient {
      * @param delegate Underlying HTTP client that performs the actual network I/O.
      * @param policies Policies that should run before the request reaches the delegate.
      */
+    @Deprecated(forRemoval = true)
     public PolicyDecoratingHttpClient(HttpClient delegate, List<HttpPipelinePolicy> policies) {
-        this.delegate = Objects.requireNonNull(delegate, "delegate cannot be null");
-        Objects.requireNonNull(policies, "policies cannot be null");
+        Objects.requireNonNull(delegate, "delegate cannot be null");
 
-        HttpPipelineBuilder builder = new HttpPipelineBuilder().httpClient(this.delegate);
-        if (!policies.isEmpty()) {
+        HttpPipelineBuilder builder = new HttpPipelineBuilder().httpClient(delegate);
+        if (policies == null || !policies.isEmpty()) {
             builder.policies(policies.toArray(new HttpPipelinePolicy[0]));
         }
         this.pipeline = builder.build();
     }
 
+    public PolicyDecoratingHttpClient(HttpPipeline httpPipeline) {
+        this.pipeline = httpPipeline;
+    }
+
     /**
      * Sends the request using the decorated pipeline without a custom {@link Context}.
      */