Commit Graph

24 Commits

Author SHA1 Message Date
klzgrad
053ff8ed17 quic: Add support for HTTP/3 CONNECT Fast Open
SpdyProxyClientSocket uses read_callback_ for both Connect() and
Read(), and its OnIOComplete() calls read_callback_, thus its fast
connect code checks read_callback_. The code was ported to
QuicProxyClientSocket without much change.

But QuicProxyClientSocket uses a separate connect_callback_ apart from
read_callback_, and its OnIOComplete() calls connect_callback_, thus
when headers are received after Connect() it doesn't need to check
read_callback_ and should always avoid calling connect_callback_.
2024-01-19 23:07:20 +08:00
klzgrad
3491205eda h2: Pad RST_STREAM frames
Clients sending too many RST_STREAM is an irregular behavior.

Hack in a preceding END_STREAM DATA frame padded towards [48, 72]
before RST_STREAM so that the TLS record looks like a HEADERS frame.

The server often replies to this with a WINDOW_UPDATE because padding
is accounted in flow control. Whether this constitudes a new irregular
behavior is still unclear.
2024-01-19 23:07:20 +08:00
klzgrad
f0b82d00ac h2: Add support for HTTP/2 CONNECT Fast Open
SpdyProxyClientSocket waits for 200 OK before returning OK for Connect.

Change that behavior to returning OK immediately after CONNECT header.

This feature is enabled by a "fastopen" header via the proxy delegate.

Design notes:

The current approach is better than the obvious TCP Fast Open style fake
Connect().

Fast Open should not be used for preconnects as preconnects need actual
connections set up. The Naive client does not use preconnects per se
(using "...RawConnect") but the user agent will use preconnects and the
Naive client has to infer that. Hence there is a need to check the
incoming socket for available bytes right before Connect() and configure
whether a socket should be connected with Fast Open. But fake Connect()
make it difficult to check the incoming socket because it immediately
returns and there is not enough time for the first read of the incoming
socket to arrive.

To check for preconnects it is best to push the first read of the
incoming socket to as late as possible. The other (wrong) way of doing
that is to pass in an early read callback and call it immediately after
sending HEADERS and then send the available bytes right there. This way
is wrong because it does not work with late binding, which assumes
Connect() is idempotent and causes sockets opened in this way to be
potentially bound to the wrong socket requests.

The current approach is to return OK in Connect() right after sending
HEADERS before getting the reply, which is to be received later. If the
reply is received during a subsequent Read() and the reply indicates an
error, the error is returned to the callback of the Read(); otherwise
the error is ignored with the connection disconnected and subsequent
Read() and Write() should discover the disconnection.
2024-01-19 23:07:20 +08:00
klzgrad
8ffda9271f h2: Reduce warnings about RST on invalid streams
Per RFC 7540#6.4:

  However, after sending the RST_STREAM, the sending endpoint MUST be
  prepared to receive and process additional frames sent on the stream
  that might have been sent by the peer prior to the arrival of the
  RST_STREAM.
2024-01-19 23:07:20 +08:00
klzgrad
ab747c7005 socket: Force tunneling for all sockets
In the socket system, only WebSocket sockets are allowed to tunnel
through HTTP/1 proxies. "Raw" sockets in the normal socket pool don't
have it, and their CONNECT headers are not sent, instead the raw
payload is sent as-is to the HTTP/1 proxy, breaking the proxying.

The socket system works like this:

- HTTP sockets via HTTP/1 proxies: normal pool, no tunneling.
- HTTPS sockets via HTTP/1 proxies: normal pool, no tunneling,
  but does its own proxy encapsulation.
- WS sockets via HTTP/1 proxies: WS pool, tunneling.

In Naive, we need the normal pool because the WS pool has some extra
restrictions but we also need tunneling to produce a client socket
with proxy tunneling built in.

Therefore force tunneling for all sockets and have them always send
CONNECT headers. This will otherwise break regular HTTP client sockets
via HTTP/1 proxies, but as we don't use this combination, it is ok.
2024-01-19 23:07:20 +08:00
klzgrad
1040a32168 socket: Allow higher limits for proxies
As an intermediary proxy we should not enforce stricter connection
limits in addition to what the user is already enforcing.
2024-01-19 23:07:20 +08:00
klzgrad
50d9f9ad68 cert: Handle AIA response in PKCS#7 format 2024-01-19 23:07:20 +08:00
klzgrad
7dfffe5ce9 cert: Use builtin verifier on Android and Linux 2024-01-19 23:07:20 +08:00
klzgrad
1ba5e45de9 cert: Add SystemTrustStoreStaticUnix
It reads CA certificates from:

* The file in environment variable SSL_CERT_FILE
* The first available file of

/etc/ssl/certs/ca-certificates.crt (Debian/Ubuntu/Gentoo etc.)
/etc/pki/tls/certs/ca-bundle.crt (Fedora/RHEL 6)
/etc/ssl/ca-bundle.pem (OpenSUSE)
/etc/pki/tls/cacert.pem (OpenELEC)
/etc/pki/ca-trust/extracted/pem/tls-ca-bundle.pem (CentOS/RHEL 7)
/etc/ssl/cert.pem (Alpine Linux)

* Files in the directory of environment variable SSL_CERT_DIR
* Files in the first available directory of

/etc/ssl/certs (SLES10/SLES11, https://golang.org/issue/12139)
/etc/pki/tls/certs (Fedora/RHEL)
/system/etc/security/cacerts (Android)
2024-01-19 23:07:20 +08:00
klzgrad
834b4aef37 libc++: Disable exceptions and RTTI
Except on Mac, where exceptions are required.
And except on Android, where rtti is required.
2024-01-19 23:07:20 +08:00
klzgrad
eb1f76ad1b base: Disable trace event
This allows builds with enable_base_tracing=false.
2024-01-19 23:07:20 +08:00
klzgrad
e290c0d661 base: Fix iwyu in file_path.cc 2024-01-19 23:07:20 +08:00
klzgrad
253c004e5a base: Don't fix Y2038 problem with icu 2024-01-19 23:07:20 +08:00
klzgrad
13be335df0 net, url: Remove icu 2024-01-19 23:07:20 +08:00
klzgrad
1b507c04ce build: Force determinism in official build
Helps build with ccache.
2024-01-19 23:07:20 +08:00
klzgrad
1d870969ec build: Disable Android java templates 2024-01-19 23:07:20 +08:00
klzgrad
3f58ff269c build: Disable build_with_chromium
The argument build_with_chromium mainly enables various tests,
data bundling, infra integration, and AFDO profiles.

AFDO can be added by other arguments.
2024-01-19 23:07:20 +08:00
klzgrad
358071f0ba url: Remove Android 2024-01-19 23:07:20 +08:00
klzgrad
325bc749dd base: Remove JNI function on Android 2024-01-19 23:07:19 +08:00
klzgrad
082f5bc311 base: Add Android stubs 2024-01-19 23:07:19 +08:00
klzgrad
2d1bf6a911 net: Add Android stubs 2024-01-19 20:03:38 +08:00
klzgrad
1694a758c0 build: Remove tests and minimize 2024-01-19 20:03:37 +08:00
klzgrad
80c1470eeb Add .gitignore 2024-01-19 10:08:33 +08:00
importer
614b0e32ad Import chromium-121.0.6167.71 2024-01-19 10:08:33 +08:00