- 18 Mar, 2024 11 commits
-
-
Dridi Boukelmoune authored
It replaces idle_send_timeout when the stream is waiting for window credits before sending more DATA frames. The idle_send_timeout will still apply to individual writes to the socket, but triggering it is considered a failure condition (this was already the case). The two loops are merged into a single one to better deal with the lack of ordering guarantees of request vs connection stream control flow window crediting. Refs #2980 Conflicts: bin/varnishd/http2/cache_http2_proto.c bin/varnishd/http2/cache_http2_send.c
-
Dridi Boukelmoune authored
-
Dridi Boukelmoune authored
This parameter needs to be somewhat high because web browsers may for example only credit streams for image resources just enough to parse metadata like width and height of a picture in order to perform layout computations and fetch resources more sensitive to latency before the effective image payloads. Conflicts: include/tbl/params.h
-
Dridi Boukelmoune authored
Conflicts: bin/varnishd/http2/cache_http2_send.c include/tbl/h2_error.h
-
Dridi Boukelmoune authored
The new meaning for h2->sess->t_idle is now the time since the last complete frame, except before the first frame where it corresponds to the creation of the session. Conflicts: bin/varnishd/http2/cache_http2_proto.c
-
Dridi Boukelmoune authored
And after issuing or receiving a goaway, stop processing frames when there are no streams left. Initially submitted as part of listen sockets management. Refs #3959 Conflicts: bin/varnishd/http2/cache_http2_proto.c
-
Dridi Boukelmoune authored
Conflicts: bin/varnishd/http2/cache_http2_send.c
-
Dridi Boukelmoune authored
Conflicts: bin/varnishd/http2/cache_http2.h bin/varnishd/http2/cache_http2_proto.c
-
Dridi Boukelmoune authored
Conflicts: bin/varnishd/http2/cache_http2_send.c
-
Dridi Boukelmoune authored
-
Dridi Boukelmoune authored
-
- 18 Dec, 2023 1 commit
-
-
Dridi Boukelmoune authored
The 503 synth and 500 minimal response status codes are too misleading in this context, where the failure is attributed to the client. Among existing 4XX status codes, this is the closest if we stretch the timeout definition to "didn't complete rapidly enough before the client went away". Conflicts: bin/varnishd/cache/cache_req_fsm.c There is no minimal 500 response on this branch.
-
- 05 Dec, 2023 3 commits
-
-
Dridi Boukelmoune authored
Since Python 3.12 unknown escape sequences trigger a warning.
-
Dridi Boukelmoune authored
The matching regular expression did not align with the substitution regular expression. Refs d2e526ce
-
Dridi Boukelmoune authored
-
- 21 Nov, 2023 2 commits
-
-
Dridi Boukelmoune authored
A VCL failure on the client side transitions to vcl_synth, except failures from vcl_synth that lead to minimal errors. The ESI transport is not allowed to reply with minimal responses so this would lead to a panic. On top of that, the vcl_req_reset feature flag emulates `return (fail)` statements when an HTTP/2 client disconnected, resulting in the same panic scenario. For sub-requests, we masquerade the fail transition as a deliver and trade the illegal minimal response for the synthetic response. Fixes #4022 Conflicts: bin/varnishd/cache/cache_req_fsm.c
-
Nils Goroll authored
Found by @Dridi Conflicts: doc/changes.rst
-
- 14 Nov, 2023 1 commit
-
-
Simon Stridsberg authored
-
- 13 Nov, 2023 3 commits
-
-
Dag Haavi Finstad authored
-
Dag Haavi Finstad authored
-
Dridi Boukelmoune authored
-
- 18 Oct, 2023 19 commits
-
-
Nils Goroll authored
we can not make the parameter const because API.
-
Dag Haavi Finstad authored
-
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
-