[master] 3c5ed76 Pass delivery abandoned does not qualify as an error

Nils Goroll nils.goroll at uplex.de
Mon Mar 5 12:51:08 UTC 2018


commit 3c5ed76fea33c98dc5464c8e8ff902172bcd8dbb
Author: Nils Goroll <nils.goroll at uplex.de>
Date:   Fri Oct 6 12:58:24 2017 +0200

    Pass delivery abandoned does not qualify as an error
    
    ... so log it under the Debug tag.
    
    FetchErrors should be actual errors which can be addressed. In this case,
    nothing is wrong in any way, the fact that we abort a fetch if we don't
    need the body is a varnish internal optimization (which makes sense, but
    comes at the cost of closing a connection).
    
    Merges #2450

diff --git a/bin/varnishd/cache/cache_fetch.c b/bin/varnishd/cache/cache_fetch.c
index 581a707..f64cfb4 100644
--- a/bin/varnishd/cache/cache_fetch.c
+++ b/bin/varnishd/cache/cache_fetch.c
@@ -449,8 +449,8 @@ vbf_stp_fetchbody(struct worker *wrk, struct busyobj *bo)
 			 * objects to be created.
 			 */
 			AN(vfc->oc->flags & OC_F_PASS);
-			VSLb(wrk->vsl, SLT_FetchError,
-			    "Pass delivery abandoned");
+			VSLb(wrk->vsl, SLT_Debug,
+			    "Fetch: Pass delivery abandoned");
 			bo->htc->doclose = SC_RX_BODY;
 			break;
 		}


More information about the varnish-commit mailing list