fix: Preserve error cause in CRT http client TLS and socket timeout failures

[joviegas]

Motivation and Context

The CRT HTTP client translates native TLS negotiation failures (CRT error code 1029)
into SSLHandshakeException and socket timeouts during connection acquisition
(CRT error code 1048) into ConnectException. Both translations build the new
exception from the message string alone and discard the underlying CRT
HttpException, so getCause() returns null and the CRT error code is lost.

Without a cause chain, callers cannot classify a TLS failure as transient
(connection interrupted during the handshake) or persistent (certificate
validation failure), and cannot write retry predicates that target one case
and not the other. The Apache, Netty, and UrlConnection clients preserve the
full cause chain for the same failure. This change fixes the CRT client to
match that behavior.

Modifications

• CrtUtils.wrapCrtException: chain the original HttpException via
  initCause in the SSLHandshakeException branch (error code 1029) and the
  ConnectException branch (error code 1048). Exception types and messages
  are unchanged, so existing callers that match on type are unaffected.
• CrtUtilsTest: add two tests asserting that each wrapped exception carries
  the HttpException as its cause.
• CrtRequestExecutorTest and CrtAsyncRequestExecutorTest: extend
  execute_httpException_mapsToCorrectException with hasRootCause(exception)
  assertions.
• SdkHttpClientSslHandshakeBehaviorTestSuite (new, in test/http-client-tests):
  shared suite that runs an HTTPS server with a self-signed certificate and
  asserts that the handshake failure surfaces as SSLHandshakeException with a
  non-null cause. Reusable by every SdkHttpClient implementation.
• CrtSslHandshakeBehaviorTest (aws-crt-client) and
  ApacheSslHandshakeBehaviorTest (apache-client): wire the new suite into
  both clients. The CRT test fails without the CrtUtils fix and passes with it.

Testing

• Added Test suite to tess all Http client

Screenshots (if appropriate)

Types of changes

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

License

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

[alextwoods]

Tiny nit on the file name here - I think .*Test.* may be too permissive - I'd lean towards making it .*Test\.java$ to enforce the name ends with Test and we don't pick up a class name that just happens to have "Test" in it.

[alextwoods]

Edit: nvm, I see the files are TestSuite. I think we should still enforce the ending, but maybe must be either *Test.java or *TestSuite.java

[joviegas]

Valid point, will make it *TestSuite.java
I think we should not add it for generic *Test.java since these classes should go in tst/ folder and not in main src.

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