]> git.ipfire.org Git - thirdparty/squid.git/commit
Ban reserved annotations in "note", "adaptation_meta" directives (#561)
authorChristos Tsantilas <christos@chtsanti.net>
Sat, 29 Feb 2020 11:37:02 +0000 (11:37 +0000)
committerAmos Jeffries <yadij@users.noreply.github.com>
Thu, 19 Mar 2020 10:49:01 +0000 (23:49 +1300)
commit8e06bc480df9c32cbd14fb92b0778025085f0d69
tree405eef174069faa63be36f03f444e6e13d3475fb
parent458c21a05101cb6db77196fa55718c9d22c954a9
Ban reserved annotations in "note", "adaptation_meta" directives (#561)

Squid defines a list of names used internally for exchanging name=value
pairs with various helpers. When Squid receives a name=value pair with a
reserved name, Squid stores it as if it was any other annotation; the
information can be checked with a "note" ACL and logged with %note. An
admin who configures a custom annotation with the same reserved name may
see strange/unexpected results such as seemingly corrupted access.log
record fields and mismatching ACLs.

Squid already prohibits reserved annotation names in
annotate_transaction and annotate_client ACLs. This change adds the
missing protection to the "note" and adaptation_meta directives.

This is a Measurement Factory project
src/Notes.cc
src/Notes.h
src/acl/AnnotationData.cc
src/adaptation/Config.cc