- 30 Oct, 2019 2 commits
-
-
Nils Goroll authored
We use the first 64bit of a sha256 as our pool id, assuming that those are safe enough against collisions. Ensure we do not fail on that assumption. This also makes it a caller error to deliberately use the same pool id for different endpoints. As this was not possible before the id change, I do not consider it a regression. Ref: ec70dbc7
-
Nils Goroll authored
Ref: ec70dbc7 struct vtp_cs was only used for comparisons during pool lookup
-
- 29 Oct, 2019 5 commits
-
-
Dridi Boukelmoune authored
If varnish closes the connection while a client is waiting for an rxsomething command you may run into this kind of scenario: *** c1 HTTP2 rx failed (fd:20 read: Connection reset by peer) *** c1 rx: stream: 0, type: DATA (0), flags: 0x00, size: 0 **** c1 s0 - no data ---- c1 Wrong frame type DATA (0) wanted SETTINGS This was already done correctly for the frame body.
-
Dridi Boukelmoune authored
Instead of having to manually document them, we can now let the RST dump figure everything out. Refs #3099
-
Dridi Boukelmoune authored
Instead, they are added where they are needed: only in error messages involving them. Refs #3099
-
Dridi Boukelmoune authored
Refs #3108
-
Nils Goroll authored
* clarify defaults * document Transient Fixes #3108
-
- 28 Oct, 2019 1 commit
-
-
Dag Haavi Finstad authored
Fixes: #3086
-
- 25 Oct, 2019 1 commit
-
-
Dridi Boukelmoune authored
Refs #2872
-
- 24 Oct, 2019 5 commits
-
-
Dridi Boukelmoune authored
-
Dridi Boukelmoune authored
Otherwise it is considered a code block instead of a paragraph where it is used.
-
Dridi Boukelmoune authored
The VSL query could in theory do that, but I don't think it was actionable without double quotes to delimit empty fields or fields containing spaces.
-
Dridi Boukelmoune authored
The test case covers the last field of Backend_health records.
-
Dridi Boukelmoune authored
When we output a VSL field containing spaces, all bets are off for VSL queries relying on that field or subsequent fields in the same record. The solution is to allow a quoted-string format for such fields.
-
- 23 Oct, 2019 1 commit
-
-
Guillaume Quintard authored
- fix centos:7 distcheck - use centos to build the dist tarball - simplify git cloning with builtin command - no need for ssh when building - tar the dsc files with the deb ones - use the regular pkg repo now that the alpine PR has been merged
-
- 22 Oct, 2019 1 commit
-
-
Dridi Boukelmoune authored
-
- 21 Oct, 2019 15 commits
-
-
Nils Goroll authored
-
Poul-Henning Kamp authored
-
Nils Goroll authored
brought up on IRC by @scn
-
Dridi Boukelmoune authored
-
Dridi Boukelmoune authored
It is only really needed for one last use case.
-
Dridi Boukelmoune authored
-
Dridi Boukelmoune authored
-
Dridi Boukelmoune authored
-
Dridi Boukelmoune authored
-
Dridi Boukelmoune authored
Inform users when the parameter we try to change depends on another parameter. This is not meant to be merged as-is, we could have a new VSB function to trim the end of the string if it matches the one passed as the second argument. In this case, we would trim the trailing newline only of the CLI response didn't overflow in the first place, adding the extra information on the same line. Alternatively we could also simply add the explanation for the dynamic bounds on the next line. Feedback welcome. Fixes #3098
-
Dridi Boukelmoune authored
-
Dridi Boukelmoune authored
-
Martin Blix Grydeland authored
req->err_code and req->err_reason are set when going to synthetic handling. From there the resp.reason HTTP field is set from req->err_reason if set, or the generic code based on req->err_code is used if it was NULL. This patch clears these members so that a value from the handling of a previous request doesn't linger. Fixes: VSV00004
-
Poul-Henning Kamp authored
-
Poul-Henning Kamp authored
-
- 18 Oct, 2019 4 commits
-
-
Dridi Boukelmoune authored
This de-duplicates logic between the two hashing directors, and in the case of vmod-shard adds a missing check for empty components in the strands that should be skipped. This breaks the hashing of the hash director, but unlike the shard director it offers no stability and results may vary depending on the health of individual clusters. For this reason only d00003.vtc needed a bit of reshuffling.
-
Guillaume Quintard authored
-
Guillaume Quintard authored
-
Guillaume Quintard authored
-
- 16 Oct, 2019 1 commit
-
-
Dridi Boukelmoune authored
-
- 15 Oct, 2019 2 commits
-
-
Dridi Boukelmoune authored
I was in the neighborhood...
-
Dridi Boukelmoune authored
This reverts commit 6233b088. Conflicts: bin/varnishd/cache/cache_req.c The conflict was introduced with the handling of h2 stream 0 pseudo request: 51127b46. This fixes a regression where timeout_idle could be circumvented by a periodic CRLF, except that it has no real consequence like holding onto a worker thread thanks to timeout_linger kicking in as expected. In addition to reverting the change, test coverage is added.
-
- 14 Oct, 2019 2 commits
-
-
Guillaume Quintard authored
This reverts commit 762fda9a.
-
Guillaume Quintard authored
-