- 18 Oct, 2023 40 commits
-
-
Dag Haavi Finstad authored
Backport of 2cfb561e and 1a406e3f Conflicts: bin/varnishtest/vmods.h configure.ac lib/Makefile.am
-
Dag Haavi Finstad authored
backport of f820f801 Conflicts: include/tbl/params.h
-
Dag Haavi Finstad authored
-
Dag Haavi Finstad authored
Only RST frames received earlier than this duration will be considered rapid. Includes backports of the following upstream doc-touchups: - 151f4d50 - 46d4f823 Conflicts: include/tbl/params.h
-
Poul-Henning Kamp authored
-
Dag Haavi Finstad authored
Conflicts: bin/varnishd/VSC_main.vsc include/tbl/sess_close.h
-
Dag Haavi Finstad authored
This adds parameters h2_rapid_reset_limit and h2_rapid_reset_period, which govern the rate of which we permit clients to reset streams. If the limit is exceeded the connection is closed. Related to: #1851 Conflicts: include/tbl/params.h
-
Asad Sajjad Ahmed authored
This should make the code easier to read and stop us from being inconsistent. Signed-off-by: Asad Sajjad Ahmed <asadsa@varnish-software.com>
-
Dridi Boukelmoune authored
-
Dridi Boukelmoune authored
The error check is not performed in a critical section to avoid contention, at the risk of not seeing the error until the next transport poll. Conflicts: bin/varnishd/http2/cache_http2_session.c
-
Dridi Boukelmoune authored
Once a client is reportedly gone, processing its VCL task(s) is just a waste of resources. The execution of client-facing VCL is intercepted and an artificial return(fail) is returned in that scenario. Thanks to the introduction of the universal return(fail) proper error handling and resource tear down is already in place, which makes this change safe modulus unknown bugs. This adds a circuit breaker anywhere in the client state machine where there is VCL execution. A new Reset time stamp is logged to convey when a task does not complete because the client is gone. This is a good complement to the walk away feature and its original circuit breaker for the waiting list, but this has not been integrated yet. While the request is technically failed, it won't increase the vcl_fail counter, and a new req_reset counter is incremented. This new behavior is guarded by a new vcl_req_reset feature flag, enabled by default. Refs #3835 Refs 61a15cbf Refs e5efc2c8 Refs ba54dc91 Refs 6f50a00f Refs b8816994 Conflicts: bin/varnishd/cache/cache_vrt_vcl.c bin/varnishd/cache/cache_vcl_vrt.c bin/varnishd/mgt/mgt_param_bits.c include/tbl/params.h
-
Dridi Boukelmoune authored
The goal is for top-level transports to report whether the client is still present or not.
-
Dridi Boukelmoune authored
It was particularly hard to follow once we reach client c3.
-
Dridi Boukelmoune authored
Not sure why 6fbb9377 does not originate from trunk.
-
Dridi Boukelmoune authored
We don't need a delay, we need to sync operations.
-
Martin Blix Grydeland authored
This parameter allows the user to choose which storage backend / stevedore that the H/2 receive buffers are allocated from. By default it uses Transient. Conflicts: bin/varnishd/mgt/mgt.h bin/varnishd/mgt/mgt_param.h bin/varnishd/mgt/mgt_param_tbl.c include/tbl/params.h
-
Martin Blix Grydeland authored
-
Martin Blix Grydeland authored
This was found lacking in our H2 implementation. Previously we would have included any padding bytes in the request body. Possibly it would have caused errors if there also was a C-L present, or more likely just corrupt request bodies. If the client sends nothing but padding bytes and ends up consuming the entire stream window with no actual request bytes buffered, the request thread side of things would not send any stream window updates. Handle this corner case by sending a window update from the session thread.
-
Martin Blix Grydeland authored
According to the spec the padding is an 8-bit field, and fields should be treated as unsigned unless otherwise specified, which it is not for any of the padding related places. Allow varnishtest to generate padding up to 255 bytes long.
-
Martin Blix Grydeland authored
-
Martin Blix Grydeland authored
-
Martin Blix Grydeland authored
-
Martin Blix Grydeland authored
-
Martin Blix Grydeland authored
-
Martin Blix Grydeland authored
This is the test case that fails if these changes aren't in tree. Note the commented out rxwinup commands that are necessary for the proper fail mode when run without the varnishtest window update changes.
-
Martin Blix Grydeland authored
With the new request body data handling, Varnish changes behaviour significantly wrt to stream window updates sent to the client. Window updates will only be sent once the data is consumed by the client through the request body VFP handling. Test cases that rely on receiving a window update to sync the H/2 stream needs to be adopted.
-
Martin Blix Grydeland authored
This makes it easier to not have to know exactly when and how many window updates to expect in a test case.
-
Martin Blix Grydeland authored
We have a strict min at the protocol default here. This is because we don't have the 'use settings only after peer ack' in place yet. If the value is lower than the protocol default, the very first stream could get a flow control error.
-
Martin Blix Grydeland authored
H2 streams waiting for request body data will timeout after timeout_idle seconds if no new data on the stream is being received. This will ensure that individual H2 streams can be reaped if there is no data received from the peer.
-
Martin Blix Grydeland authored
-
Martin Blix Grydeland authored
This implements stream data handling using a buffer between the H/2 session thread and each stream thread. This is needed to avoid head of line blocking on the session socket when a data frame is received for a stream thread that is not yet ready to receive it. The buffer used will have to be as large as the send window the peer expects at the time the stream is opened. This will typically be 65535 unless the h2_initial_window_size parameter has been changed. Stream window updates will then be issued only once data is removed from the buffer by the request body being consumed from the request handling thread, limited in size to what space is then available in the buffer. Conflicts: bin/varnishd/http2/cache_http2_proto.c
-
Martin Blix Grydeland authored
The H/2 session thread does have a VSL buffer already set up, but the 'wrk->vsl' pointer was not set. This caused issue for e.g. LRU_NukeOne() as it wants to log. Set the buffer for the duration that the worker is dedicated as an H/2 session thread.
-
Dridi Boukelmoune authored
And use that for logging purposes when a successfully opened h2 session ends. RX_JUNK is still the default session close reason when existing reasons aren't accurate enough. Fixes #3393
-
Dridi Boukelmoune authored
In the same fashion as include/tbl/params.h for legibility.
-
Martin Blix Grydeland authored
-
Martin Blix Grydeland authored
-
Martin Blix Grydeland authored
-
Martin Blix Grydeland authored
Conflicts: bin/varnishd/storage/storage_simple.c
-
Martin Blix Grydeland authored
This is an API for getting an arbitrary buffer through the stevedores. The stevedore in question may then deploy LRU nuking or other measures to control resource usage.
-
Dridi Boukelmoune authored
-