From 2bf636de988315a3f3166c1d37c2b0072b1e889d Mon Sep 17 00:00:00 2001 From: Christos Tsantilas Date: Tue, 11 Jun 2013 13:04:48 +0300 Subject: [PATCH] Fix icap_log and log_icap acls checking The HTTP reply related acls does not check for icap_log --- src/adaptation/icap/Xaction.cc | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/adaptation/icap/Xaction.cc b/src/adaptation/icap/Xaction.cc index 8f82586f77..41fc63404a 100644 --- a/src/adaptation/icap/Xaction.cc +++ b/src/adaptation/icap/Xaction.cc @@ -550,7 +550,11 @@ void Adaptation::Icap::Xaction::tellQueryAborted() void Adaptation::Icap::Xaction::maybeLog() { if (IcapLogfileStatus == LOG_ENABLE) { - ACLChecklist *checklist = new ACLFilledChecklist(::Config.accessList.icap, al.request, dash_str); + ACLFilledChecklist *checklist = new ACLFilledChecklist(::Config.accessList.icap, al.request, dash_str); + if (al.reply) { + checklist->reply = al.reply; + HTTPMSGLOCK(checklist->reply); + } if (!::Config.accessList.icap || checklist->fastCheck() == ACCESS_ALLOWED) { finalizeLogInfo(); icapLogLog(alep, checklist); -- 2.47.2