GHSA-xr57-gcx8-52hf
AsyncHttpClient sends origin credentials to the proxy on the plaintext CONNECT request
Quick fix
GHSA-xr57-gcx8-52hf — org.asynchttpclient:async-http-client: upgrade to the fixed version with the command below.
# pom.xml: bump <version>3.0.12</version> for org.asynchttpclient:async-http-clientDetails
### Impact When a request uses an HTTP proxy to reach an HTTPS origin, the client opens the tunnel with a plaintext CONNECT sent to the proxy before any TLS exists. On affected versions the origin's preemptive credentials were added to that CONNECT. A Basic realm sent `Authorization: Basic base64(user:pass)` to the proxy in the clear, and NTLM, SPNEGO or Kerberos realms sent their token. The proxy, and anyone who can read the client to proxy hop, saw credentials that were meant only for the origin.
### Affected versions * 3.x: up to and including 3.0.11 * 2.x: up to and including 2.16.0
### Patches Fixed in 3.0.12 on the 3.x line and in 2.16.1 on the 2.x line. The origin Basic or Digest Authorization is no longer put on the CONNECT; it is added to the tunnelled request once the tunnel is up. The per-connection NTLM, Kerberos and SPNEGO token is likewise no longer attached on the tunnel path; it never reached the origin there in any case, and the challenge flow still negotiates it inside the tunnel.
### Workarounds Do not use preemptive origin authentication together with an HTTP proxy, or reach the origin without a CONNECT proxy.
### Details NettyRequestFactory added the origin Authorization to every request it built, including the CONNECT, and NettyRequestSender did the same for the per-connection NTLM or SPNEGO token in sendRequestWithNewChannel. Both now skip the CONNECT.
Note that 3.0.12 is itself affected by a separate issue, GHSA-rqf5-2wxv-rjf4, where a Digest challenge the client cannot read downgrades to Basic and sends the password in cleartext. Upgrade to 3.0.13 to pick up both fixes.
Are you affected?
Enter the version of the package you're using.
Affected packages
3.0.0Fixed in: 3.0.12# pom.xml: bump <version>3.0.12</version> for org.asynchttpclient:async-http-client2.0.0Fixed in: 2.16.1# pom.xml: bump <version>2.16.1</version> for org.asynchttpclient:async-http-clientReferences
- https://github.com/AsyncHttpClient/async-http-client/security/advisories/GHSA-xr57-gcx8-52hf[WEB]
- https://github.com/AsyncHttpClient/async-http-client/pull/2234[WEB]
- https://github.com/AsyncHttpClient/async-http-client/commit/9dba5ac988b7e750551f59b2eab550b60ac8a0d6[WEB]
- https://github.com/AsyncHttpClient/async-http-client[PACKAGE]