dbopen configure test typo

Added "self" refcounted Pointer parameters to StoreIOState's STRCB, STFNCB,
and STIOCB callback functions.  This is so that the StoreIOState
refcount doesn't go to zero during one of these callbacks.

Do the dbopen compile test proper to see if -ldb is needed

Moved STRCB, STFNCB, STIOCB typdefs to inside StoreIOState class itself.
This should allow us to use these types as parameters in the callbacks.

Fix COSS so it works with my recent StoreIOState changes.

Bootstrapped

Do a full compile test of dbopen to see if -ldb is needed

From 2.5: Properly quote wbinfo arguments, just in case.

2.5 commit log reads "Minor fix for Samba-3", but later changes made it
work anyway..

Merged storeIOState and StoreIOState into a single StoreIOState
class.  Previously, StoreIOState was just a refcounted storeIOState.

I had concerns that mixed use of refcounted and non-refcounted
StoreIOState may be leading to cases where the object gets referenced
via the old-style pointer after the refcounted version had its count
go to zero and the memory was freed.  This came about while
investigating bugzilla #1465.

removed StoreIOState-related prototypes from protos.h, and removed
some typedefs from typedefs.h.

Modified STFNCB (storeSwapInFileNotify, storeSwapOutFileNotify) and
STIOCB (storeSwapOutFileClosed, storeSwapInFileClosed) so that they
no longer take StoreIOState arguments.  Their "data" arguments
easily lead to the corresponding StoreIOState.  The only thing we
really lose here is the ability to assert that "data->sio" equals
the passed StoreIOState.

temporary hack to handle case where body_reader becomes NULL
in http module before sendRequestEntity gets called.

Bootstrapped

Fix http_range test to run again.

Minor cleanup of the meta TLV format to not add the incorrect STORE_META_END
at the end of the list (list ends by it's size). STORE_META_END has changed
a number of times over the history, and we have never cared about it but some
external tools (i.e. purge) apparently beleives what they read in the docs..

Made stream outputting more consistent in Mem::PoolReport.  Use a
const char * for the column delimiter instead of repeating "\t "
on each line.  Always put the delimiter at the end of the (source code) line.

Try to clean up cachemgr 'mem' output.  Don't need to double up on
'%' now that we're not using printf.  Use std::setprecision() on some
fields so they don't show up as "1e+01" etc.

struct lock_ctrl_t and callback type SIH seem to be unused anywhere
in the code.

Converted storeRelease() to a StoreEntry method.

Fully deprecated storeLockObject().  Everything now directly calls the
StoreEntry->lock() method.

Fully deprecated storeUnlockObject().  Now everything directly uses
the StoreEntry unlock() method.

put correct function names in debugging statements

This debugging statement should not be at level 0.

The patch for bug #1499 included debugging statments where printf tokens
did not match time_t type on FreeBSD.  The time_t variables are now cast
in debugging calls.

Bug #1475: Squid does not need to be limited by FD_SETSIZE when using kqueue/epoll.

Technically poll doesn't need to be limited either, but is due to the delay
pool hooks still using fd_set...

Inspired by the patch by Gonzalo Arana

Bug #1501: ftp downloads may last up to read_timeout
by Gonzalo Arana

Bug #1499: external acls are somewhat broken
by Gonzalo Arana

Give I/O threads a smaller stack.

Added usage of configure results

Syslog is not available on all platforms

Need SquidTime for squid_curtime

Bug #1489: add syslog priority and facility to access_log

Patch provided by Gonzalo Arana: Thanks !

Bug# 1430: feature: multi-line directives in squid.conf

Patch provided by Gonzalo Arana. Thanks !

Bug #1347: Modify squid MIB to incorporate squid.conf visible_hostname

Patch provided by John Horne, thanks !

Don't disable assert on PURIFY

Bug #1553: Memleak in HTCP client code

Forward port of 2.5 changes.

Fix Cygwin build errors.

Bug #1532: Mime icons are not displayed when viewing ftp sites when
visible_hostname is a short hostname (without domain).

Forward port of 2.5 changes.

Bug #1558 (cosmetic): cachmgr leaks a stdio buffer

Bootstrapped

bug: ICAPAccessCheck should use cbdataReference and cbdataReferenceValidDone
in case the upstream ClientRequestContext goes away during processing.

need to include <netinet/in.h> on FreeBSD for definition of sockaddr_in

Refcount::Pointers don't need getRaw() when comparing to NULL

bug: clientAbortBody should not try to close the connection
if it is already being closed.

Need config.h before any system heaers

Bootstrapped

Forgotten to dist SquidTime.h

Fix copy-paste install dependenices when separating cachemgr & squidclient
from src

Bootstrapped

Move squidclient and cachemgr from src to tools

Need SquidTime for squid_curtime

Put squid.h before <iosfwd> to avoid FD_SETSIZE redefined errors
with 'make check'

Bootstrapped

Unwind HTCP dependencies

Bootstrapped

util.c depends on assert.c

Bootstrapped

Also need a generic xassert to go with out assert, to be used by helpers
etc.

Broke out assert define into it's own include. Relied upon by MemPool

Take 2 of the FD_SETSIZE redefined fix.. eleminate the need of squid.h
completely (was only needed for definition of NULL which isn't needed in C++)

Fix include order to make sure config.h is always first (indirectly via squid.h)

Bug 1224 / 1530: Crash related to failed PUT/POST requests where
body processing may close the connection "in the middle" of client
side request processing.

Bug #1530: Make "foo" a more intuitive error message by just using
assert instead.

Need SquidTime.h to squid_curtime

Removed the HttpHeaderEntry constructor that created a "blank" object.
It created problems with the parse() method because id had to be set
to something, but none of the http_hdr_type values were really appropriate,
not to mention they either triggered assertions or confused the statistics.

Now parse() is a static method that calls new HttpHeaderEntry after
parsing has been successful.

Bug #1531: squid crashes short after starting

When HttpHeaderEntry was converted to a class with methods I added a
constructor that took no arguments. It initialized id to HDR_BAD_HDR.
If header parsing fails, HDR_BAD_HDR triggers an assertion in
the destructor.  As a workaround id is now initialized to HDR_OTHER.

These .cc files need to #include the new SquidTime.h

bootstrapped

More decoupling work:
 Add a URL and URLScheme class to allow separating the url parsing logic from
 HTTPRequest logic which are currently bi-dependent.
 Add HttpRequestMethod class to replace the enum method_t, start using it.
 Split squid_curtime out from globals and tools, so you can get access to time
 without dragging in other dependencies.

Fix diskd debugging to print the correct unlink function name, either
unlink or truncate.

added .dirstamp

converted httpHeaderEntry functions to class methods

bugfix: Diskd has a loop that puts Squid to sleep for small amounts of
time when the queues size reach a threshhold.  The time-to-sleep is
supposed to double each time through the loop, but struct timeval delay
was initialized inside the loop, rather than outside, which meant it
always slept for 1 microsecond.

Clean up some recently-added debugging messages.

A new fix for the bug when FwdState reference count goes to zero
during a re-forward attempt.

client_side_reply.h doesn't need to #include forward.h any more.

Bootstrapped

Bug #767: Problem making Squid 3.0 PRE3 64 bit

Final fix for %llu problem on HP Tru64.

Backed out change where clientReplyContext held a FwdState::Pointer.
That change was to fix problems with re-forwarded reqeusts where
the FwdState refcount would go to zero.  But it also introduced a new
bug where errors would not be sent to clients because the refcount
did NOT go to zero.

Bootstrapped

Forgotten CXXFLAGS for --with-large-files

Bootstrapped

Added missing size checks from 2.5

- Forward ported from 2.5 the updated large file build environment support (bug #437)
- Cosmetic cleanup

Bootstrapped

Converted most other httpHeader*() functions to HttpHeader class methods.

Converted most other httpHeader*() functions to HttpHeader class methods.

Converted most other httpHeader*() functions to HttpHeader class methods.

Fixed testHttpRequest with --enable-referer-log & --enable-useragent-log and
when GNUregex is needed.

Unbreak compiling on g++3.3

converted httpHeaderClean() to HttpHeader::clean()

Fix for recent "clientReplyContext keeps a FwdState refcount" patch.
I missed the more common case where fwdStart() was called.

new feature: squid -k restart

It allows the built-in parent "watcher" process to automatically
and immediately restart Squid.

-k restart uses SIGTTIN.  It is almost the same as -k shutdown
except that it sets the exit status to 1 (instead of 0).  Since the
exit status is non-zero, the parent (watcher) will start squid again
immediately.

-k restart is not supported on systems that dont have SIGTTIN.

Two bugfixes

- ftpSendPasv() should only close the data socket, rather than call
  transactionComplete().

- If ICAP service selection fails, we need to call call
  entry->replaceHttpReply() so that the HTTP headers get written
  to the StoreEntry buffer.

minor bugfix: immediately shutdown interprocess socket to 'pinger'
if send() returns ECONNRESET.

Fix for : assertion failed: forward.cc:99: "err"

This assertion was triggered when FwdState tries to re-forward a
request.  The first request fails, but can be retried.  If the first
ServerState gets destroyed before the second ServerState gets
created, the FwdState refcount goes to zero.   The FwdState destructor
has the above assertion because it expects an ErrorState for the
StoreEntry with no content.

This condition can easily happen because the second ServerState
isn't created until the origin server connection is established.
ie:

http.cc(1318) transactionComplete FD -1 this 0x8a430b0
fwdComplete: re-forwarding 504 http://www.example.org/
fwdConnectStart ...
commConnectHandle: ...: COMM_INPROGRESS
httpStateFree: ...
forward.cc(87) FwdState destructor starting
assertion failed: forward.cc:99: "err"

The clientReplyContext class now keeps a refcounted pointer to
FwdState.  This was done so that FwdState refcount doesn't go to
zero during a "reforward" between the time when the initial ServerSide
gets destroyed and before the second attempt ServerSide gets created.

We don't really want to keep FwdState around until the client is
finished.  As an future optimization we can remove the client side
reference around the time when the ENTRY_FWD_HDR_WAIT bit gets
cleared.

cosmetic: convert two static functions to ConnectStateData class methods.
Also updated function name in one debugging statement.

bugfix: must reset hdr->mask to 0 after cleaning HttpHeader.

Backing out last patch (httpHeaderGetSize()).  The problem is elsewhere.

Bugfix for assertion in httpHeaderFindEntry().  The caller,
HttpMsg::hdrCacheInit() was assuming that the content length header
is always present in the HTTP reply.

To fix I copied httpHeaderGetSize() from squid-2 code.  For now
this version of httpHeaderGetSize() is basically the same as
httpHeaderGetInt(), except that it checks to see if the header
exists.  If not, it returns -1.

Forward port patch #1518 - strip NT domain component from usernames.

Bootstrapped