[master] a73e6a5b8 For BackendClose, also log the short tag as with SessClose
Nils Goroll
nils.goroll at uplex.de
Fri Mar 1 14:09:06 UTC 2024
commit a73e6a5b8ee4543f82cafeb3ce5e377963ca6ed3
Author: Nils Goroll <nils.goroll at uplex.de>
Date: Mon Jan 29 16:56:09 2024 +0100
For BackendClose, also log the short tag as with SessClose
this is for consistency, to simplify parsing and to reduce the amount of
data logged with VSL.
Motivated by #4042
diff --git a/bin/varnishd/cache/cache_backend.c b/bin/varnishd/cache/cache_backend.c
index 61be0aa27..7b273c5b7 100644
--- a/bin/varnishd/cache/cache_backend.c
+++ b/bin/varnishd/cache/cache_backend.c
@@ -240,7 +240,7 @@ vbe_dir_finish(VRT_CTX, VCL_BACKEND d)
bo->htc->priv = NULL;
if (bo->htc->doclose != SC_NULL || bp->proxy_header != 0) {
VSLb(bo->vsl, SLT_BackendClose, "%d %s close %s", *PFD_Fd(pfd),
- VRT_BACKEND_string(d), bo->htc->doclose->desc);
+ VRT_BACKEND_string(d), bo->htc->doclose->name);
VCP_Close(&pfd);
AZ(pfd);
Lck_Lock(bp->director->mtx);
diff --git a/include/tbl/vsl_tags.h b/include/tbl/vsl_tags.h
index 378916cab..b5a3596c0 100644
--- a/include/tbl/vsl_tags.h
+++ b/include/tbl/vsl_tags.h
@@ -155,7 +155,7 @@ SLTM(BackendClose, 0, "Backend connection closed",
"The format is::\n\n"
"\t%d %s %s [ %s ]\n"
"\t| | | |\n"
- "\t| | | +- Optional reason\n"
+ "\t| | | +- Optional reason, see SessClose for explanation\n"
"\t| | +------ \"close\" or \"recycle\"\n"
"\t| +--------- Backend display name\n"
"\t+------------ Connection file descriptor\n"
More information about the varnish-commit
mailing list