Summary: Fix iterator-conflict in the name Splay::end
Keywords:

Fix iterator-conflict in the name Splay::end

Summary: Update test scripts.
Keywords:

Update test scripts for minor bitrot.
Make mem_hdr_test scricter on memory use.

Summary: Fix outgoing_address and tos.
Keywords:

Fix outgoing_address and tos.

Summary: request_flags incorrectly initialised nocache_hack.
Keywords:

request_flags incorrectly initialised nocache_hack.

Summary: ACL tweaks.
Keywords:

Move aclCheckFast to ACLChecklist::fastCheck(), and adjust callers.
Move aclChecklistCreate into the ACLChecklist source.
Move the common code leading into matchAclList into the top of matchACLList.

Fix persistent connections to origin server class peers if the port
of the peer differs from the request.

From 2.5: log "Reconfiguring Squid Cache .." rather than "Restarting
Squid Cache.." on "squid -k reconfigure".

Bootstrapped

Fix SQUID_MAXFD % 64 test (was using % 63 ...)

Summary: signed correctness for the mem pool idle tracker.
Keywords:

signed correctness for the mem pool idle tracker.

Bootstrapped

From 2.5: Make SQUID_MAXFD always an multiple of 64 to guarantee there is no rounding
errors in bitfield arithmetics of the fd_set macros. For example GLIBC has
bugs causing a fd_set overflow if the size is not an even multiple of a
machie word (64 bits on 64 bit platforms)

From 2.5: winbind update to
  * Don't complain with errors in cache.log on normal restarts
  * Give winbind a little time to start if not running already

Documentation update to document all the accelerator options in https_port.

also cleaned up http_port options to follow the same style as https_port

Bug #767: Problems compiling Squid on 64 bit Alpha

cosmetic code changes to make pointer asserts type safe

the same problem quite likely exists in other parts of the code.

Bug #769: assertion failed: ftp.cc:1150: "fd == ftpState->data.fd"

Did not deal with COMM_ERR_CLOSING correctly on aborted requests.

From Pawel Worach:

HEAD errorpage.cc compile fix
Update this assert too now that the MemObject is gone.

dup the path string first - the l1l2 parsing routine spits out a log
message which relies on it.

Summary: Don't double free refcounted object on store swapout failures.
Keywords:

Don't double free refcounted object on store swapout failures.

Summary: Allow -DPURIFY to build.
Keywords:

Allow -DPURIFY to build.

regenerated HTML to be in synch with SGML code

Reorder members to match constuctor calls to make GCC happy

Bootstrapped

Summary: the always-finsh-compile-testing splay fixup.
Keywords:

The splay fix previous was incomplete..

Summary: Initialise splay node's.
Keywords:

The SplayNode template did not initialise it's members.

From Reuben:
configure comments updates to remove Linux-2.4 dependency for Netfilter.
Netfilter is Linux-2.4+ and also Linux-2.2 with netfilter patch..

Bootstrapped

Summary: Fix cbdata debugging memory leak.
Keywords:

Implement placement new and placement delete for cbdata. Prevents history leaking memory, without needing ugly manual destruction etc.
Make cbdata::type typesafe.

Summary: Class cleanup - resulting from addressing a latent segfault.
Keywords:

Move httpMsgIsPersistent delcaration to HttpHeader
Rename http_version_t to HttpVersion, and make a class.
Remove httpBuildVersion - use HttpVersion constructor.
Move HttpVersion to it's own file.
Move HttpStatusLine to it's own file and make a class.
Move AccessLogEntry to it's own file, and make a class.
Remove memset() use against AccessLogEntry.

Summary: Coss bugfixes.
Keywords:

Remove unused prototypes from store_dir_coss.cc.
Fixup a couple of bugs in the new coss option parsing.
Bugfix debug options to allow debugging of the parser.

Summary: Fix race in UFS code.
Keywords:

UFS had a race leading to a refcount triggered free within a method.

Summary: Windows Merges - dup2 and debugging.
Keywords:

These are 2 residual Windows port merge patches:

- main.cc: On Windows signal() can't be used for debugging purpose: added
support for native  debug
- comm.cc: On Windows dup2() can't work correctly on Sockets, the work
around is to close the destination socket before call them.

Regards

Guido

bootstrapped

Summary: failure to detect <sys/mount.h> on NetBSD
Keywords:

On NetBSD-current, squid3-HEAD fails to detect sys/mount.h.
The attached patch to acinclude.m4 fixes it.

* need to include <net/if.h> before, not after, <netinet/ip_fil.h>

* neet to include <sys/time.h> before <netinet/ip_fil.h>.

For some reason, these problems do not prevent "configure" from detecting
<netinet/ip_fil.h>, but do prevent configure fron detecting <sys/mount.h>.

Summary: Tweak the COSS update.
Keywords:

Parse coss options.
Fix (hopefully correctly) some signed < unsigned comparisons.

Found a old STAT_GRAPHS patch laying around in my Squid source directories.
This should make the STAT_GRAPHS define result in valid graphs again..

ported COSS enhancements from squid-2.5 code (dated ~ July 25-29, 2003).

Summary: Backport a fix for the diskd assertion inProgressIOs == 0.
Keywords:

Backport a fix for the diskd assertion inProgressIOs == 0.

Remove the link to Glenns LDAP auth page.. It is no longer of interest to
users of this helper as the helper has evolved far beyond what it once looked
like, and also his page appears to be offline..

Bug: "access_log none" segfaults Squid on startup

Bug #733: No explicit error message when ncsa_auth (squid user) can't access passwd file

Created section "Other changes mostly of interest to developers" and moved
corresponding entries there.

SGML syntax corrections (bad characters etc)

Indent

Removed a few entries which was news in the 2.5 cycle and not really news
for 3.0.

Summary: Stop reply generation after parsing failures.
Keywords:

If we cannot parse the response to generate a reply, stop reply generation. Prevents a SIGSEGV in httpBuildVersion.

Bootstrapped

Summary: Test for -fhuge-objects in configure. (Bug #744)
Keywords:

Test for -fhuge-objects in configure. (Resolves bug $744)

Non-intrusive but significant performance improvement when Squid is running
as a https reverse proxy (ssl enabled peers), allowing Squid to reuse the
same SSL session.

Summary: gethostname definition on Solaris broken.
Keywords:

The C++ compiler detection seems to allow the native declarations of gethostname to function on Solaris. Copy that logic into protos.h so that we don't conflict, but still have our declaration should it be needed.

Summary: debug format string portability.
Keywords:

pid_t is not an int on all platforms - fixed by moving to the new debugs macro.

Bootstrapped

Summary: Squid_endian.h was broken - bug 738.
Keywords:

Mismatched parentheses in the definition of bswap16 in line 87
od squid_endian.h.  Try the attached patch.
- apb

Bug 718: accel fails with DNS error if cache_peer name= not in /etc/hosts

If we don't find anything valid in pconnPop, don't try and return an fd.

Stupid bug brought to you by: robertc, adrian

bump version

bootstrapped

Summary: Use SQUID_C[XX]FLAGS throughout configure.
Keywords:

Where we are setting defines etc. that are required to build squid correctly, set them via SQUID_C[XX]FLAGS, not C[XX]FLAGS, so that the user can override the user-related flags (i.e. optimisation / profiling / debugging) without breaking the squid build.

Summary: Prevent pushing persistent connections with outstanding writes.
Keywords:

Adrians' recent post to squid-dev looks like it's triggered by persistent connections being pushed inappropriately... this change adds an assert if a persistent connection is pushed while a write is outstanding.

bootstrapped

Summary: Dist recent new files.
Keywords:

Dist recent new files.

Summary: Update test-suite for recent type changes.
Keywords:

Update test-suite for recent type changes.

bootstrapped

Summary: BUGFIX: ufs auto-add logic was broken.
Keywords:

* src/Makefile.am referenced unused variable STORE_SOURCES.
* configure.in incorrectly set STORE_LINKOBJS when ufs was not selected.

This also fixes bug #746.

Summary: Swallow request bodies even when denying access.
Keywords:

* Assert when destroying requests with body connections.
* Move ClientBody class definition to client_side.h
* Encapsulate comm_close calls from ClientSocketContext.
* When closing a client socket, if there is a body, swallow it.
* When swalling a request body, if the connection needs to close, do so.

Bug #718: accel with nonstandard cache_peer port uses standard port

The recent fix for vhost acceleration and persistent connections
confusing certain servers did a little bit too much, accidently using
the port of the URL rather than the cache_peer in case of origin class
peers.

Bug #743: "#ifdef HTTP_VIOLATIONS" should be "#if HTTP_VIOLATIONS"

Bug #743: Error when compile on FreeBSD 4.8-stable OS

"#ifdef HTTP_VIOLATIONS" should be "#if HTTP_VIOLATIONS"
because it is set to "0" in include/autoconf.h

Bug #736: ICP dynamic timeout algorithm ignores multicast

The algorithm that calculates the timeout for a set of ICP
queries ignores multicast neighbors.  It also ignores the
expected number of replies because "*exprep" is always set
equal to parent_exprep + sibling_exprep.

This patch adds multicast group counts to the expected number
of ICP replies.  It also sets the query timeout to the
multicast peer rtt (x2) if there were no parents query.  If
there are parents, the expected parents timeout is used.

Summary: Really get the corrent accesslog syntax in cf.data.pre
Keywords:

Really get the corrent accesslog syntax in cf.data.pre

the I need more sleep checkin

Summary: Apply the correct cf.data.pre patch for %<sS and %<sH
Keywords:

Apply the correct cf.data.pre patch for %<sS and %<sH

Summary: Incorrectly astyled cf.data.pre
Keywords:

Incorrectly astyled cf.data.pre

Summary: Add %<sS and %<sH to custom log formats.
Keywords:

Bugfix to access logging - the last request in a connection was racing with releasing of the store entry.
Add new high offset and available offset counters.

Summary: MORE!!! MSVC Fixes.
Keywords:

Don't use initialiser syntax for char const *'s.
Don't use member address in initialisation lists.

Summary: Cachemgr page length - Guido.
Keywords:

Hi,

On Windows native port, cachemgr parses the squid output in TEXT mode, with
a wrong resultant Content-Length in html pages, see this thread for details:

http://www1.it.squid-cache.org/mail-archive/squid-dev/200306/0044.html

The attached patch force cachemgr to work only on BINARY mode and contents
a cosmetic change.

Regards

Guido

Summary: Gcc-3.3-recent fix from ismail donmez.
Keywords:

Hi,

Patch attached!

Regards,
/ismail

Bug #663: Username not logged into ACCESS.LOG in case of /407

The Basic auth module did not implement the direction call correctly,
indication authentication scheme failure on unsuccessful helper lookup
rather than "completed".

Note: This API and it's interactions with checklist->auth_user_request &
request->auth_user_request looks overly complex for the job. Should be
possible to simplify this a lot I think.

Summary: Tweak install instructions.
Keywords:

Tweak install instructions - add -z requirement.

Summary: Final MSVC fixups.
Keywords:

* Don't reuse loop variable in errorPageId.
* Remove all uses of request_t - replace with HttpRequest.

Summary: Fixup remaining MSVC issues.
Keywords:

Addresses -

protos.h:
c:\work\nt-3.0\src\protos.h(443) : warning C4190: '<Unknown>' has C-linkage
specified, but returns UDT 'String' which is incompatible with C
         c:\work\nt-3.0\src\squidstring.h(77) : see declaration of 'String'
c:\work\nt-3.0\src\protos.h(444) : warning C4190: '<Unknown>' has C-linkage
specified, but returns UDT 'String' which is incompatible with C
         c:\work\nt-3.0\src\squidstring.h(77) : see declaration of 'String'
c:\work\nt-3.0\src\protos.h(445) : warning C4190: '<Unknown>' has C-linkage
specified, but returns UDT 'String' which is incompatible with C
         c:\work\nt-3.0\src\squidstring.h(77) : see declaration of 'String'
c:\work\nt-3.0\src\protos.h(446) : warning C4190: '<Unknown>' has C-linkage
specified, but returns UDT 'String' which is incompatible with C
         c:\work\nt-3.0\src\squidstring.h(77) : see declaration of 'String'
c:\work\nt-3.0\src\protos.h(447) : warning C4190: '<Unknown>' has C-linkage
specified, but returns UDT 'String' which is incompatible with C
         c:\work\nt-3.0\src\squidstring.h(77) : see declaration of 'String'

protos.h:
c:\work\nt-3.0\src\protos.h(740) : error C2526: 'storeCreate' : C linkage
function cannot return C++ class 'RefCount<class storeIOState>'
c:\work\nt-3.0\src\protos.h(741) : error C2526: 'storeOpen' : C linkage
function cannot return C++ class 'RefCount<class storeIOState>'

typedefs.h:
c:\work\nt-3.0\src\httprequest.h(56) : warning C4099: 'HttpRequest' : type
name first seen using 'struct' now seen using 'class'
         c:\work\nt-3.0\src\typedefs.h(202) : see declaration of 'HttpRequest'

ACLRequestHeaderStrategy.h
c:\work\nt-3.0\src\aclrequestheaderstrategy.h(88) : error C2059: syntax
error : 'constant'

ACLStrategised.h
c:\work\nt-3.0\src\aclreplyheaderstrategy.h(88) : error C2059: syntax error
: 'constant'

ACLReplyHeaderStrategy.h:
c:\work\nt-3.0\src\aclreplyheaderstrategy.h(87) : error C2059: syntax error
: 'constant'

Summary: Remove use of FreeObject template function for MSVC compatability.
Keywords:

Remove use of FreeObject template function for MSVC compatability.

Fixing abuse of struct in_addr in auth_digest authenticate to
allow building on MinGW again..

(reported by Guido)

Summary: Forward port removing MemObject->fd.
Keywords:

Bug #94 is solved by removing the MemObject->fd variable. Much of this was already in 3.0, but the remainder wasn't.

Summary: Fix range offsets when the full object is retrieved.
Keywords:

* Extend StoreIOBuffer to provide Range details.
* Extend ClientSocketContext::lengthToSend to be offset aware and adjust throughout.
* ClientSocketContext::packRange - tidy up the calling interface, make private, and skip unwanted data at the beginning as well as the end of ranges.

detemplatize splay comparisons for MSVC

Bug #726: File src/fs/diskd/store_io_diskd.cc doesn't compile

use debugs() macro instead of debug() to take advantage of automatic
typesafeness (no format strings which may mess things up)

really fix enum use problem in digest for MSVC

update to squid_endian.h from Alan Barrett

Correct data channel management in case of errors while establishing the data channel - hno - bug #700

Bootstrapped

NetBSD defines bswap16() and bswap32().  The re-definitions in [squid]/include/ntlmauth.c cause compile errors.  I'll send a patch to the squid-dev list soon. - Alan Barrett (bug #724)

Alan Barrett's fix for inconsistent bootstrapping on some platforms

small clarification in the accel changes to try to get people on the
cache_peer track.

Summary: Remove the unneeded 'deleteSelf' idiom.
Keywords:

Remove the unneeded 'deleteSelf' idiom.

Cut out unwanted regex interfaces from GNUregex.c
 - emacs related stuff
 - BSD 4.2 regex interface
 - old GNU regex interface
leaving only the POSIX interface which is what we use

update for changed eventAdd linkage

store_null.cc compilation errorsstore_null.h not disted  - bug #727

Squid will not compile with --enable-epoll only (disable select and poll) Bug #728

fix compiling on cygwin without the win32 service enabled - bug #731

backout pam_auth suid change

From email:
> > MemObject.cc
>
>Looks like another limitation. taking the first one: there is a for_each
>template that is explicitly for dlink_list. You might try
>for_each<StoreClientsStats>(clients, statsVisitor);

Rigth, now MSVC++ is happy:

Hi,

On Windows (native and Cygwin) and OS/2, when running
UFSSwapDir::closeTmpSwapLog() in store_dir_ufs.cc, Squid can be fail if the
target file for a rename operation was already deleted.

Because xrename() already try to remove the destination target on Windows,
a previous unlink() is not needed.

This patch extended the native Windows xrename() behaviour to Cygwin and
OS/2 too and remove not needed unlink().

Regards

Guido