fix(http): trim nonProxyHosts entry whitespace

[zoewangg]

Motivation and Context

nonProxyHosts / no_proxy values are pipe- or comma-separated lists, and users commonly write them with spaces after the separator, e.g.:

export no_proxy="example.com, *.foo.com, 10.0.0.0/8"
-Dhttp.nonProxyHosts="example.com | *.foo.com"

The shared parser (SdkHttpUtils) split these lists but did not trim the resulting entries, so an entry with a leading (or trailing) space, e.g. " *.foo.com", was stored with the space as part of the host name. It then never matched the real host, so the host was routed through the proxy instead of bypassing it. This silently defeated no_proxy for any list written with the conventional comma-space style.

Modifications

• Fix (behavior): SdkHttpUtils.extractNonProxyHosts now trims each entry (.map(String::trim)) after splitting, before lowercasing and the glob-to-regex rewrite. This shared parser feeds the four Java-regex clients (netty-nio, apache, apache5, url-connection) on the system-property (http.nonProxyHosts) and environment-variable (no_proxy/NO_PROXY) resolution paths, so the fix applies to those four uniformly.

• Documentation (no behavior change): added documentation

• CRT scope (why no CRT changelog entry): CRT already trims nonProxyHosts entries via its own path (rawNonProxyHosts() → ProxyUtils.splitToGlobTokens) as of the merged CRT wildcard change, so this PR's shared-parser trim does not change CRT behavior — it affects only the four Java-regex clients. The changelog therefore has four entries (one per regex client) and no CRT entry.

  No public API surface changes; builder-supplied nonProxyHosts semantics are unchanged (documented, not altered).

Testing

• Shared cross-client suite: added a comma-space no_proxy case to ProxyConfigCommonTestData (drives HttpProxyTestSuite for all five clients) asserting each entry resolves with surrounding whitespace trimmed.
• Netty routing: added ProxyWireMockTest.proxyConfigured_hostInCommaSpaceNoProxyEnv_doesNotConnect, verifying a host listed in a comma-space no_proxy bypasses the proxy end to end.
• Unit: added SdkHttpUtilsTest cases for comma-space entries, surrounding whitespace on a single entry, and the no-whitespace regression (unchanged behavior preserved).

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)

Checklist

• I have read the CONTRIBUTING document
• Local run of mvn install succeeds
• My code follows the code style of this project
• My change requires a change to the Javadoc documentation
• I have updated the Javadoc documentation accordingly
• I have added tests to cover my changes
• All new and existing tests passed
• I have added a changelog entry. Adding a new entry must be accomplished by running the scripts/new-change script and following the instructions. Commit the new file created by the script in .changes/next-release with your changes.
• My change is to implement 1.11 parity feature and I have updated LaunchChangelog

License

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

[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.