backing out revision 1.715 getRaw() patch.

remove ugly getRaw()'s  They are not necessary when comparing pointer
values.

remove ugly getRaw()'s from refcounted pointer class

Rework FtpStateData::transactionComplete() so that it normally gets called
*after* the control connection is closed, rather than before sending
the QUIT command.  This should avoid the race condition where the FTP
control socket is closed before we get any data back from the ICAP server.

Add proxy authentication to the list of allowed headers in paranoid anonymization

Fix HttpReply leak due to failure to unlock in this class

when logging the client request, al.reply might be NULL.  If so, don't
link to it.

Fixed gcc 4.1 build problem

Bug #1485: logformat debugging messages

Patch provided by Gonzalo Arana.

Cleaned up HttpReply handling in client_side_reply code.

ACLChecklist should lock/unlock the HttpReply.

clientReplyContext had a "holdingReply" member which I had a hard
time figuring out.  My guess is that it was just a way to avoid leaking
the HttpReply.  I've replaced it with a more permanent HttpReply
which is now locked and unlocked as necessary.

I am changing debugging levels for these frequently-printed messages.
They make full debugging virtually worthless because they overwhelm
the other messages.

For now 'squid -kdebug' changes debugging to 'ALL,7' instead of 'ALL,10'

Bug #1513 Some 206 responses logged incorrectly

Fortunately the AccessLog class already had the HttpReply. We just need
to use it and for good meaure, lock and unlock it.

Moved some common code into the ClientHttpRequest constructor.

Replaced the single-line wrapper function storeEntryReplaceObject
with direct calls to StoreEntry->replaceHttpReply();

Removed static requestLink() and requestUnlink() methods.  Replaced them
with HTTPMSGLOCK() and HTTPMSGUNLOCK() macros.

An ICAP server is allowed to list multiple methods in an options
response.  Previous code complained if there were multiple methods.

We should pass HttpStateData's orig_request to ICAP, rather than
request, which will be different (and generate wrong URLs) if forwarding
to a parent cache.

Make ftpWriteCommand a method of FtpStateData

The 'ICAPClientRespmodPrecache' pointer in HttpStateData was moved to
ServerStateData.

ServerStateData::doIcap needs to be protected with #ifdefs

The purpose of this change is to add ICAP RESPMOD support for FTP responses.

I created a "ServerStateData" class which has common elements of both
HttpStateData and FtpStateData.  It becomes a base class for both
of them.  ICAP now uses the ServerStateData methods.

bootstrapped after adding src/Server.{cc,h}

The purpose of this change is to add ICAP RESPMOD support for FTP responses.

I created a "ServerStateData" class which has common elements of both
HttpStateData and FtpStateData.  It becomes a base class for both
of them.  ICAP now uses the ServerStateData methods.

Corrected port of patch for Bug #577. Issue found by Gonzalo Arana

change _ftp_flags from unsigned int:1 to bool.

Remove whitespace in function args leftover from s///g;

Update MARA Systems sponsor text now that they no longer actively sponsor
Squid.

64-bit compatibility

Workaround for newer OpenLDAP libraries requiring LDAP_DEPRECATED in order
to allow access to the "good old standard" LDAP C-API functions..

Kill unused nreadfds/nwritefds statistics in comm_poll, eleminating one
more reference to fd_set. The delay pool hooks still uses fd_set however
so comm_poll is not yet FD_SETSIZE neutral.

More C++ conversions for ftp.cc

cleanup in constructor

FtpStateData constructor was all wrong

Started converting ftp.cc to use more C++ class methods

protect from double inclusion with #ifndef directives

Fixed ICAP memory leak.  ICAPClientRespmodPrecache no longer has
to "give up control" of the HttpReply since it is now lock-counted.

Fix lock-counting bug caused by mixing references with this lock counting
technique.

Also added locks for HttpReply in ICAPAccessCheck::ICAPAccessCheck()

Moved HttpRequest lock/link counter to the base HttpMsg class.  This
makes it easy to also lock-count HttpReply instances.

Removed HttpReply::swapOut(), which was a trivial wrapper around
StoreEntry::storeEntryReplaceObject().  Just call storeEntryReplaceObject()
instead.

Removed HttpReply::absorb() which was a hack to deal with the fact that
HttpReply's were not lock-counted.

Bug #1479: wrong timezone declaration for 64 bit Irix
The declaration of timezone is different between 32 and 64 bit Irix

Addedd .cvsignore

A static definition of w_space here is no more needed

My recent ClientBody changes introduced a memory leak.  This is awkward
because ConnStateData and ClientBody have references to each other.
ClientBody has a RefCount::Pointer to ConnStateData, which must be NULLed
so the ConnStateData destructor will be called.

fix for new HttpRequest

Rename previsouly private HttpRequest:initXX() to initHTTP() and use
it in urlParse().

Making HttpRequest more class-like.  Removed some non-class functions such
as requestCreate() and requestDestroy() and fixed up the clean/reset mess.

Typo in #ifndef #define pair

Bootstrapped

Bootstrapped after adding src/ClientBody.{h,cc}

ConnStateData and ClientBody was a little confusing.  There was some duplication
with ClientBody members also appearing in ConnStateData.body.   They have been
merged now and ClientBody lives in its own .{cc,h} files.

Don't allow ICAP connection reuse if there is a timeout on the socket

Management of {adapated,virgin}->data->header was becoming a problem.
One side might delete the header before the other side was done with it.
This patch moves deletion of ->data->header to MsgPipeData class so that
it goes away only when the MsgPipe goes away.

incorrectly committed new debugging with wrong levels

C++ typo: extraneous `int' ignored

Minor debugging changes to StoreEntry constructor/destructor

Fixed a couple of bugs with ClientHttpRequest::doCallouts()

- The "no_cache" rule check was never happening because flags.cachable
  was always 0 at that point.  clientInterpretRequestHeaders() must
  be called before doing the "no_cache" checks.

- ClientRequestContext::checkNoCache had an extra cbdataReference() call
  which caused a cbdata leak.

- Converted the *_done flags to bool (instead of single-bit ints).

Changed debugging locations for MemObject constructor and destructor
so they print at the start of the function.

In the destructor there is no need to check that _reply is set because
nobody else deletes it.

ICAPClientRespmodPrecache::noteSourceStart() should know whether
or not to expect an HTTP body and call the appropriate next note
function.

Mark requireOpenAndActive() as a static function

Bugzilla #1463

IdentLookup::checkForAsync() should only set asyncInProgress(true)
when actually doing the non-blocking ident lookup.

Commiting various debugging statements that have been useful in following
the flow of an ICAP transaction.

A comment about this place where we link request to a mem_obj

The HttpStateData::abortAdapting() method should always free up
the HttpStateData, not just when entry is empty.

Added some timeout debugging for ICAP transactions.  Since ICAPXaction
doesn't know anything about its derived classes, also added a
HttpMsg::firstLineBuf() method to be used in the debugging of timeouts.

This patch adds sorting to cachemgr:mem output to make it easier to find
classes/objects that leak memory.

using requestLink on adapted->data->header fixes an HttpRequest memory
leak with malformed ICAP/HTTP responses.

HttpRequest::reset() should preserve link_count

Bootstrapped

New file forward.h is missing from make dist

Forgot to commit http.h after changing HttpStateData constructor

convert an old-style cast to a C++ static cast

Moved most of httpStart() into the HttpStateData constructor.

Fix HTCP after moving httpBuildRequestHeader() to HttpStateData class.

Oops, forgot to commit changes to this file along with http.cc

Converting more http.cc functions to class methods

Converted some functions here to static wrappers plus class methods.

Fixed an ICAP bug when reading an HTTP message body and the adapted
response does not need a body.

ICAP fix: don't call fwd->complete() in HttpStateData::abortAdapting().
Let the forwarding module decide what to do when its destructor gets
called.

Change 'fd=%d' to 'FD %d' in debugging statments to be consistent with
the rest of Squid code.

Bugs from recent FwdState change with --enable-icmp

Bugs from recent FwdState change with --enable-cache-digests

Bug in FwdState destructor.  Go back to using a temporary FD value
when closing server_fd.

Added forward.h when converting FwdState to a C++ class

Converted FwdState to a C++ class to take advantage of features
such as refcounting.

Bootstrapped

Bootstrapped

Get rid of LDAP libaray references. Not needed or wanted here..

Properly fix snapshot diff

Fix cleanup of old snapshots again. Got broken some months ago..

Fix diff generation

imported current mksnapshot script from 2.5

Simple debugging message addition and change

Found and fixed a case where HttpMsg::parse() returned false but
forgot to set the error parameter.

Fix digest helper split

Radius Digest authentication extension draft

Bootstrapped

Fix external acl helper selection again

silence automake warning about underquoted macor

Correct digest_ldap_auth program name

Bootstrapped

Default to build "all" available helpers of each selected feature
(i.e. build basic auth helpers if basic scheme is enabled etc)

Dist the Digest LDAP helper