Shuffle HttpRequestMethod.h to libsquid-http.la

Add missing http/forward.h

Cleanup HttpRequestMethod.h useless lines

Fix http/Http1Parser.h wrapper macro

Merged from trunk rev.13197

Rename HttpParser as Http::Http1Parser

Make HttpParser parse version directly into AnyP::ProtocolVersion object

There appears to be no need for the protocol version object to be outside
the parser. We can simplify the processing code by parsing directly into
the object from the I/O buffer.

TODO: allow parsing of non-HTTP protocol labels.

Shuffle HttpParser to libsquid-http as Http1Parser

Drop HttpParser::Pointer::getRaw() uses

Detatch useless build dependencies on HttpParser.h

Convert API of HttpParser::parseRequest() from tristate to boolean

Move HttpParserParseReqLine() legacy function to HttpParser method

Inline and document the HTTP message size macros

Drop HttpParser MSGDODEBUG

Documentation tweak

Bug 3985: 60s limit introduced by balance_on_multiple_ip breaks bad IP recovery

Alternative fix if necessary would be to calculate a dynamic timeout
value based on number and timeouts for connections and forwarding.

However, as noted in the bug report the address markings are reset every
TTL when the stored RR are replaced. This meets the criteria for periodic
retries of marked-BAD IPs. Also if all IPs are marked BAD they all get
un-marked and retried. So for now just remove the TTL hack.

Fix \-unescaping in quoted strings from helpers

strwordtok() was unescaping '\' values in unquoted tokens. This causes
problems with NTLM helper response user\DOMAIN values which are sent as
a un-quoted un-escaped value.

Restrict un-escaping of '\' to only occur when inside "quoted" words.

WCCPv2: fix assertion 'Cannot convert non-IPv4 to IPv4' on FreeBSD

FreeBSD does not set the from-address on recv() if there was an error
reading the packet. This can result in Ip::Address assertion if the
error is not checked for before the IP address is mapped into Squid
internal format.

Fix missing cast in rev.13162

Added missing header in client_side_reply.cc for clang

Fix missing include in rev.13190

Rename SBuf::find_first_of and find_first_not_of to camelCase.

Added send_hit and store_miss squid.conf directives
to control caching of responses using response info.

The existing "cache" directive is checked before Squid has access to the
response and, hence, could not use response-based ACLs such as http_status.
Response-based ACLs may be essential when fine-tuning caching. Squid Bug 3937
(StoreID can lead to 302 infinite loop) is a good use case.

Updated old "cache" directive documentation to provide more information, to
help folks distinguish the three related directives, and to polish for
clarity.

TODO: Support lookup_hit and possibly deprecate/remove "cache".

Merge CharacterSet

Reverted buggy change on CharacterSet::operator+=, documentation touches

Merged from trunk

Merged from trunk

Merged from trunk

Merged from trunk

Add unit tests for method parsing update

Fix missing include in rev.13186

Improved CharacterSet documentation.

Merged from trunk

Using std::copy_if for CharacterSet::operator +=

Clarified documentatio for CharacterSet; moved CharacterSet::add and constructor to .cc file

Move CharacterSet::operator += to .cc file; change SBuf::find_first_not_of to match STL

HTTP/1.1: method names are case-sensitive

RFC 2616 states that method names are case sensitive, but the Squid
parser has been accepting them case-insensitive.

Enforce case-sensitive behaviour when performing strict HTTP parse.

NOTE: avoid HTTP_VIOLATIONS since there is no normative MUST/SHOULD
involved in the specification texts.

Bug 3980: FATAL ERROR due to max_user_ip -s option

Moved CharacterSet's backing storage to std::vector<uint8_t> to gain speed at the expense of space.
Improved CharacterSet's constructor
Made CharacterSet's name parameter optional with default value.

Moved CharacterSet as forward-declaration in SBuf.h

Implemented CharacterSet-based SBuf::find_first_of and find_first_not_of

Silence SBufDetailedStats stubs

Merge SBufList

Clarified comment in SBufAlgos.h, improved coding standards adherence

Implemented Tokenizer::token and its unit test

Implemented more Tokenizer methods and unit tests

Fixed initialization problem in testCharacterSet
Removed debug code in testTokenizer
Completed testTokenizerPrefix

Broken: try and figure out CharacterSet::operator +=; more Tokenizer::prefix unit tests

Fix linker errors "relocation R_X86_64_32 against .rodata"

... "can not be used when making a shared object; recompile with -fPIC"

Caused by some libraries built without libtool in certain environments.
NP: there are other libraries in Squid built without libtool but they
    appear not to be having these linker issues at this time.

Also, fixes "undefined symbol eui64_aton" uncovered by these changes.

Regression in URL helper API

The backward compatibility logics in redirect.cc are not working as
intended on redirection URLs due to the presence of '=' in the URL and
how the key=value name parsing is performed.

A typical redirection URL looks like:
  http://example.com/?url=http://www.example.net/

and 3.4 has a parser that splits tokens at '=' unconditionally and then
passes the bits as a key and value to the redirector logics which
complains that it does not understand the answer of the URL redirector.
Or treats is an an unknown key=value with no redirection URL.

Either case is handled as a no-redirection result from the helper.

This limits the key names to alphanumeric, hyphen and underscore
characters. Valid URL responses contain characters outside this set and
should no longer be interpreted as keys regardless of the '=' character.

Fix missing test stub in rev.13178

Bug 3806: Caching responses with Vary header

Set sslcrtvalidator_children concurrency option default value to 1

Merged from trunk

Fix infinite-loop in CharacterSet::operator+=()

The uint8_t type value cannot represent the value 256, which is why we
use it as the loop terminator and an iterator of a size larger than
8-bit with a down-cast to prevent the vector accidentally being grown by
an overflow.

Reimplemented prefix, implemented prefix matching unit test

Fix tokenizer unit tests scaffolding

implemented untested Tokenizer::find_first_not_in and Tokenizer::prefix

Fist stub of tokenizer unit tests

interim: better adhere to CharacterSet API, first stubs of Tokenizer

Merged from trunk

Update CharacterSet.h

Add parser/libsquid-parser.la

Also, basic class CharacterSet definition

%tS logformat code, part3

Fix build when GCC 4,4.3 used

Prep for 3.4.1

Release notes: spotlight key_extras feature

Release Notes for Squid-3.5

Fix 3.4 Release Notes typo

Add key_extras to proxy authentication schemes.

The key_extras value is a "quoted string" with logformat %macro support. It is
appended to request line for the authentication helper.

Example usage:
   auth_param basic key_extras "thePort:%>lp"
   auth_param digest key_extras "LocalIP=%>la:%lp"

This is a Measurement Factory project.

%tS logformat code, part2

Fix build on 3.HEAD-amd64-CentOs-icc

Fixed formatting
Moved ToSBufList from SBufList.h/cc to wordlist.h/cc
Fixed some misnamed parameters in SBufAlgos.h

%tS logformat code

This patch add the %tS logformat code to represent master transaction start time
in <seconds>.<milliseconds> format, similar to the existing access.log "current
time" field (%ts.%03tu).

Also allow formated values for adaptation_meta values.

This is a Measurement Factory project

Added missing header in SBufAlgos.h

Prep for SBufList merge from stringng

Bug 3589: intercepted and ICAP modified request using a cache_peer

Removed spurious source files from src/icmp/Makefile.am

Bug 3935: Invalid pointer dereference when peeking at origin server certificate

We must check request->clientConnectionManager pointer for validity before
dereferencing it.

This is a Measurement Factory project

Documented that ACLs should be refcounted.

Refcounting would avoid ACL destruction problems (when some ACLs use other
ACLs or when an ACL goes async during reconfigure) and may even reduce
problems with AclMatchedName.

Destroy ACLs in the reverse order of creation to avoid destruction segfaults
during reconfiguration.

Group ACLs created later may use other ACLs created earlier. A group ACL must
be deleted first so that its AclDeleter can safely access registration status
(and avoid double deletion) of the ACLs it uses. Since ACLs are remembered (in
Config.aclList) using a singly-linked list, it is difficult to change their
deletion order.  Instead, we change their listing order from FIFO to LIFO.

Portability: sleep() is sometimes a macro

sleep() is sometimes defined as a macro. Such as in Squid compat library.
Use another name for the sleep starting method in class Ipc::UdsOp

  This is an iCelero Project

Windows: fix compile errors in WinSvc.cc

Fix defined by unused symbol errors in WinSvc.cc

Also, set the service error mode to prevent Windows Error Reporting
dialog boxes appearing.

  This is an iCelero Project

Portability: std::string:npos is not always appropriate for String::npos

On some systems such as Windows MinGW the signed/unsigned property of
std::string::npos does not match up with the Squdi String class size_type
signed/unsigned property. Resulting in compiler signed vs unsigned
mismatch errors when compiling.

  This is an iCelero project.

Portability: refresh_pattern requires regex

The GnuRegex portability wrapper is required explicitly by
RefreshPattern.h on some systems which do not include regex.h implicitly
via other paths.

  This is an iCelero project

librfcnb: portability fixes

Add #include wrapper protection for headers missing on Windows MinGW.

  This is an iCelero Project

OpenBSD portability fix in DiskThreads

Prep for 3.3.11 and 3.4.0.3

Protect aclIsProxyAuth() debugging from NULL names (via NULL AclMatchedName).

Related to Bug 3972 fix in r13156.

Bug 3972: Segfault when getting the deny info page ID after a reconfigure

Older ACL code was using a stale AclMatchedName value. More recent code resets
the AclMatchedName global to NULL to avoid leaking the stale value, but that
may crash strcmp() in aclGetDenyInfoPage(). Long-term, the global should be
removed, of course.

Support libecap v1.0, allowing asynchronous adapters and eCAP version checks.

After these changes, Squid can support eCAP adapters built with libecap v1.0,
but stops supporting adapters built with earlier libecap versions (due to API
changes). The new libecap version allows Squid to better check the version of
the eCAP adapter being loaded as well as the version of the eCAP library being
used. This should help with migration to libecap v1.0.

Expose [running] main event loop as a global so that modules can add engines.

Added missing stub_lipbicmp.cc to testUfs

Receive annotations from external ACL helpers

This completes the annotation support for common helper interfaces by
making custom key=value pairs sent by external ACL helpers in to
NotePair objects and attaching to the active request.

The other side of this - sending values to the helper is deferred until
the helper format can be converted to logformat codes.

Add missing ipcache.cc stub in testUfs

Implement SBuf and MemBlob size distribution stats

Cleanup: convert helper allocation from cbdataAlloc to new

The helper_server and helper_stateful_server classes are already
CBDATA_CLASS2 defined classes with new/delete operators and the free
operation already handled by delete(). However teh allocation was not
using new() and thus not calling the class constructors properly.

Drop obsolete class StringArea

class StringArea was always intended to only be a temporary measure
before SBuf was completed and merged. That has now happened.

Bug 3782: Digest authentication not obeying nonce_max_count

Fix other issues in rev.13143

Fix const-correctness in rev.13143

Missing include from rev.13143

Bug 3970: max_filedescriptors disabled due to missing setrlimit