- 16 Apr, 2019 3 commits
-
-
Dridi Boukelmoune authored
I also decided to lowercase semantic patch file names, and to not mislead people about the WS_Reserve change being a vmod-only thing.
-
Poul-Henning Kamp authored
-
Poul-Henning Kamp authored
Add Project Governance info.
-
- 15 Apr, 2019 20 commits
-
-
Nils Goroll authored
Closes #2971
-
Nils Goroll authored
-
Nils Goroll authored
-
Nils Goroll authored
-
Nils Goroll authored
-
Nils Goroll authored
We handle the error case just above
-
Nils Goroll authored
-
Nils Goroll authored
Fixes #2946 in the sense that we want to treat a cacheable backend synth like any other object and kill the stale object it replaces. We do not replace the stale oc when - abandoning the bereq - leaving vcl_backend_error with return (deliver) and beresp.ttl == 0s - there is a waitinglist on this object because in this case the default ttl would be 1s, so we might be looking at the same case as the previous Thank you @mbgrydeland for the review and spotting a mistake.
-
Nils Goroll authored
Historically, all initializers called from VGC_Load() would handle failures individually (return (1) from VGC_Load) or not fail at all. Now that we got VRT_fail(), it makes sense to generalize. The upcoming use case is to properly handle failures from VRT_new_backend_clustered().
-
Nils Goroll authored
WS_ReserveSize() does not leave the workspace reserved when the reservation fails, so WS_Release() must be called for retval > 0 only. Besides the debug string, it is identical to WS_Reserve() except for assert(bytes > 0); Follow-up #2967
-
Nils Goroll authored
This works with gcc 6.3.0 and clang 3.8.1-24 The test for __GNUC__ is deliberately simple and might not catch all compilers which would potentially support deprecation marks. While more specifics could be added, the aim is to raise awareness with developers and we consider it quite unlikely that anyone does not compile with one of the main stream compilers at all.
-
Poul-Henning Kamp authored
-
Nils Goroll authored
Seen fail on travis: ** c1 2.7 === expect resp.status == 200 **** c1 2.7 EXPECT resp.status (200) == "200" match ** c1 2.7 === expect resp.http.Age == 1 ---- c1 2.7 EXPECT resp.http.Age (2) == "1" failed
-
Dridi Boukelmoune authored
To keep track of the number of times pipe_sess_max kicked in. This one can be lazily updated.
-
Dridi Boukelmoune authored
Optionally, it can limit the number of concurrent pipe transactions as they each monopolize one worker thread.
-
Dridi Boukelmoune authored
This allows users to keep track of the number of ongoing pipe transactions at any time.
-
Dridi Boukelmoune authored
Refs #2948
-
Dridi Boukelmoune authored
Closes #2948
-
Poul-Henning Kamp authored
Fixes #2959
-
- 10 Apr, 2019 13 commits
-
-
Federico G. Schwindt authored
-
Federico G. Schwindt authored
-
Federico G. Schwindt authored
-
Federico G. Schwindt authored
-
Federico G. Schwindt authored
-
Dridi Boukelmoune authored
-
Nils Goroll authored
With 32ef5dce, this test also cancels backend requests and thus may prevent the server from finishing writing the reponse body Ref #2964
-
Nils Goroll authored
I had overlooked the fact that for a canceled request we might only read parts of the body. Interestingly, this is only exposed on the vtest ARMs. Ref #2964
-
Dridi Boukelmoune authored
This is the intent that was not obvious to me when I first reviewed this change.
-
Nils Goroll authored
Ref #2963
-
Nils Goroll authored
as requested by @Dridi
-
Nils Goroll authored
for anything but return (deliver) Fixes #2963
-
Nils Goroll authored
-
- 09 Apr, 2019 2 commits
-
-
Poul-Henning Kamp authored
-
Poul-Henning Kamp authored
-
- 08 Apr, 2019 2 commits
-
-
Poul-Henning Kamp authored
-
Nils Goroll authored
... to un-confuse the interface Notes on changes from WS_Reserve(): * Removed the first WS_Assert because all we change is ws->r and we got a specific assert on it. * it follows from PAOK(ws->e) && PAOK(ws->f) in WS_Assert() that PAOK(ws->r) && PAOK(b), so we remove the PRNDDN() Closes #2967
-