Handled Surface API review comments

[joviegas]

Handled Surface API review comments

Motivation and Context

This change adds support for configuring a local address in the Apache5 HTTP client, bringing it to feature parity with the Apache HTTP client.

Modifications

1. Added local address configuration to Apache5HttpClient:

   • Added localAddress field to the DefaultBuilder class
   • Added localAddress(InetAddress) method to the Builder interface
   • Added setter method setLocalAddress(InetAddress) for bean-style configuration
   • Modified addProxyConfig() to handle local address configuration when no custom route planner is provided

2. Updated client name:

   • Changed CLIENT_NAME from "Apache5" to "Apache5Preview" to identify Preview in user agents

3. Added test coverage:

   • Created Apache5HttpClientLocalAddressOnBuilderFunctionalTest to test local address configuration via builder
   • Renamed existing test to Apache5HttpClientLocalAddressRoutePlannerFunctionalTest for clarity
   • Updated MetricReportingTest to expect the new client name

4. Package name consistency:

   • Updated .brazil.json to use consistent casing: AwsJavaSdk-HttpClient-Apache5Client-Preview

Testing

License

• I confirm that this pull request can be released under the Apache 2 license

[sonarqubecloud]

Quality Gate failed

Failed conditions
63.2% Coverage on New Code (required ≥ 80%)
18.1% Duplication on New Code (required ≤ 3%)
B Security Rating on New Code (required ≥ A)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

[github-actions]

This pull request has been closed and the conversation has been locked. Comments on closed PRs are hard for our team to see. If you need more assistance, please open a new issue that references this one.