Changed HERE macro to be a true nop, clarified documentation, and marked is as deprecated.
Fixed syntax bug in comm.cc unearthed by the macro-to-function change.

Bug 3279: Polish: reference bugzilla record to prevent duplicate reports

Bug 3329: Polish: reference the bugzilla number in cache.log to prevent duplicates

Fixed issue introduced by debugs() change

Modify debugs() to automatically include file, line and function name for high-detail cache log lines.

SourceFormat Enforcement

Multiple certificate_db/ssl_crtd fixes

- Try to update the index file in all cases the database modified
- The find operator in database should not modify the database. Currently
  if an entry is expired, ssl_crtd removes the cert file but does not
  update the index file.
  rows. Currently we are using the new operator.
- Fix a small memory leak when remove entries from database: A row object
  removed from TXT_DB indexes but never released. This patch:
     * Use OPENSSL_malloc and OPENSSL_free to allocate/release memory for
       TXT_DB rows. OpenSSL SDK assumes that always allocated using these
       functions.
     * Add code in Ssl::CertificateDb::Row destructor to correctly release
       a TXT_DB row.
     * Add the sq_TXT_DB_delete and sq_TXT_DB_delete_row functions which
       removes a row from TXT_DB indexes.

This is a Measurement Factory project

Fix broken ssl_crtd helper interface

With the patch r12422 which added support for OK/ERR/BH  helper response codes,
the ssl_crtd interface become broken. The ConnStateData::sslCrtdHandleReply
ssl_crtd reply handler, expects a response code included at the beggining of
the returned message. This is not true after the changes.

This patch add a parameter in Ssl::CrtdMessage constructor to define a message
as "ssl_crtd reply" or "ssl_crtd request" message. For ssl_crtd reply messages
the message parser does not expect helper response code.

This is a Measurement Factory project

Polish: improve const correctness on Squid vector<> implementation

This update allows == and != operators to be used with const_iterator
and removes the need for many implicit conversions safely.

Bug 3659: read_timeout problem with HTTPS

Cleanup: remove dead testNull unit test

Add SEND_BH macro to helper API definitions

SourceFormat Enforcement

Portability: Move testRefCount to src/

It seems the automake dependencies on stub_*.o are still not working well
and ICC requires several stubs in order to link libbase.la.

This also adds new stubs for MemBuf.h and cbdata.h APIs.

Polish: replace several assert(isOpen(fd))

Reduces by over half the number of asserts which can be confused as
cache.log bug reports just says "assertion isOpen(fd)" in comm.cc

Fix HelperReply(const HelperReply&) is private

GCC 4.1 and possibly others use copy-constructor when an instance is
created in a function parameter list using copy by reference.
We must create such objects as loal variables and pass to the function
for best efficiency.

Fix HelperReply(const HelperReply&) is private

SourceFormat Enforcement

Support OK/ERR/BH response codes from any helper

Updates the helper reponse callback API from using char* buffer to a
HelperReply object storing teh response code, a blob buffer, and
pointer to the responding helper 'server' (if stateful).

* the helper I/O read handler is updated to parse the result code off
the start of the helper response as is currently done for channel-ID.
The callback handlers are altered to use the HelperReply::status instead
of parsing it off themselves individually.

* the remaining I/O read buffer is stored in a MemBuf and callbacks are
updated to use it via the method other().

* the responding helper-server is stored into the HelperReply object and
stateful helper callbacks are combined into the same API as stateless.
The callback handlers are updated to use HelperReply::lastserver instead
of function parameter.

After this patch the helper response format is:
  [channel-ID] SP [result] [ [SP] blob] <terminator>

'SP' being one octet \0x20 character.

The behavour changes expected from this is that all helpers are now able
to send OK/ERR/BH states. Although the handlers for some helpers will
deal with the new states as unknown response. None of the bundled
helpers have yet been altered to make use of this changed potential.

TODO:
* implement key=value parser for the blob area of the format, and update
handlers to use the HelperReply API to retrieve them.
* upgrade helpers to make use of new response format

Fix spelling and compile issues in CommIO

Fix broken include in urllogin ACL

Polish: update the RefCount API a bt and split Lock out

* Shuffle RefCount.h and its unit-tests into src/base/

* Reworks struct Refcountable_  into class LockableObject in its own header
 + changing the reference counter accessors to a lock()/unlock() names
 + some minor symbol updates of code directly utilizing the RefCountable_
   members

With this we can begin the process of replacing our multiple different
implementations of the reference-counting pattern using LockableObject.

No code changes have been made. Just symbol polishing.

TODO: update the unit-tests for refcounting to use CPPUnit

Bug 3189: AIO thread race on pipe() initialization

Ported: urllogin ACL from squid 2.7

Fix ecap after Http::MethodType change

SourceFormat Enforcement

note option

Add forgotten Notes.cc and Notes.h file

note option

This patch adds note option to squid.conf. It is similar to
adaptation_meta but is applied after all adaptation and before logging.
Values of key/value pair notes can be logged using %{key}note macros.

    note key value acl ...
    logformat myFormat ... %{key}note ...

This option may be initially used to log custom information about the master
transaction. For example, an admin may configure Squid to log which "user group"
the transaction belongs to, where "user group" will be determined based on a
set of ACLs and not [just] authentication information.

From user point of view, adaptation_header sets/implies meta (i.e.,
setting adaptation_meta is sufficient to be able to log it using
%note) but the note option itself (if any) is evaluated later,
so it has no effect on ICAP headers.

This is a Measurement Factory project

Fixed mistakenly reverted test

Refactored tools/purge conffile reading to c++-streams

Polish: Http::MethodType upgrade

Convert the METHOD_* enum into a class with accessors to determine
method string/image, idempotency, safety (as per RFC 2616 terminology).

* creates the initial http/ directory and http/libsquid-http.la
  library as per SourceLayout needs
  - libsquid-http is chosen to prevent symbols mistakes with libhttp
    which may exist on some systems under the libhttp or http/libhttp name.

enum changes:

* migrates _method_t to Http::MethodType and makes it a enum with
  auto-generated strings array provided by the new library.

* the enum list is extended to contain all methods in the HTTPbis WG
  new method registry
  - documentation now references that registry and HTTPbis draft
    document listing methods

HttpRequest class changes:

* cacheable() changed to maybeCacheable() accessor
  - to reflect that it determines NON-cacheability with certainty, but
    when returns TRUE it can still be overridden by Http Reply details
    and/or local environment details.
  - updated to cache the complex calculation result.

HttpRequestMethod class changes:

* The new isIdempotent() and isHttpSafe() accessors are altered
  slightly from the original code to match the RFCs. The existing code
  produces incorrect true/false values on some methods.

* new accessor shouldInvalidate() which produces true if the request
  method alone is enough to determine an RFC SHOULD invalidate condition.
  - purgesOthers() accessor is still present with the old behaviour, but
    for better caching we should convert the code to using
    shouldInvalidate()

* new accessors respMaybeCacheable() which produces true if the
  request method alone is enough to determine an RFC MAY cache condition.
  - replaces isCacheable() accessor which did not produce any certainty
    as implied by the name, and was producing incorrect assessments anyway
    for some methods.
  - it is more of a test whether *Squid* is known to be able to cache
    the response to this method, than whether the RFC permit it.
  - There are several cases where it should produce true according to
    the RFCs but isCacheable() was doing false due to bugs or missing
    feature support. These cases are now documented with reasons and things
    to test or fix before they can be made to return true.

* the remainders of obsolete extension_methods directive code is dropped

Global httpCachable(method) function dropped - replaced by
HttpRequest::maybeCacheable() accessor which does better cacheability
checks.

... and there are a lot of METHOD_FOO -> Http::METHOD_FOO symbol-only
changes fluffing out the patch size.

ext_lm_group_acl: Add missing rfc1738.h include

Remove Bungled warning on missing component directives

* Make Squid display a ERROR notice and ignore squid.conf directives
  which belong to components disabled at build time instead of throwing
  up 'FATAL: Bungled' message and aborting.

* Polish the 'requires X' message text for better display of cases like
  'requires MS Windows' and 'requires Linux'

* Add 'requires X' message for --enable-auth, --enable-eui, and
  --enable-ssl-crtd which were missing.

Fix missing predefines in PeerDigest.h

Add constructore/destructor for Mgr::ActionPasswordList

Missing constructor for Mgr::ActionPasswordList resulted in invalid values
for next pointer and segmentation fault parsing multiple cachemgr_passwd
directives.

Missing destructor resulted in memory leaks on reconfigure and shutdown.

Prep for 3.2.3 and 3.3.0.1

Remove the refresh_pattern ignore-no-cache option

With the HTTP/1.1 Cache-Control:no-cache compliance upgrade this option became
obsolete. Its main use-case was to force caching of responses which contained this
control. Revalidation is now happening and no-cache objects are cached by default.

The one remaining effect of ignoring no-cache is to prevent no-cache from allowing
authenticated responses being cached. Which is the opposite of its usually expected
result and allowing that change of behaviour would be counter productive.

HTTP/1.1: Cache-Control compliance upgrade

trunk rev 11361 converted Cache-Control header from using a single mask
bitmap (shared by request and response) to separate CC header objects in
the request response. This conversion contained several regressions like
the one bug 3670 reports.

This patch:
 * documents HttpStateData::cacheableReply() clarifying the overall
   method action and what each individual check it doing.
 * resolves several visible regressions, including bug 3670.
 * extends the caching to handle the "no-cache" controls as per HTTP/1.1
   (MAY store, but MUST revalidate before use).
 * extends the caching for several lesser known cases of "MAY store"
   exemptions handling authenticated transactions.
 * removes an old hack handling Pragme:no-cache (undefined in HTTP/1.1)

One side effect of now caching transactions utilizing "no-cache" is that
hacks around Pragma:no-cache are reduced to only having any effect when
Cache-Control is absent. Reducing their performance cost. And now require
--enable-http-violations is built in since HTTP/1.1 states that
response Pragma are not defined and thus SHOULD be ignored.

SourceFormat Enforcement

basic_sspi_auth: several compile issues on MinGW

* Fixes build issues with usage() function definition removing the use of
  global my_program_name

* Fixes double definition of safe_free()

* Removes the DEBUG macro. Helper API uses a -d run-time flag now.

Windows: revert broken attempt at removing #define macro wrappers

"using Squid::foo" style of defining when namespace functions to use when
there is a clash appears not to work on MinGW. We are forced to retain
the old #define macros.
This resolves "error: call of overloaded ... is ambiguous"

Update a few functions parameters to POSIX compliant APIs using socklen_t.
This resolves "error: invalid conversion from 'socklen_t*'"

SourceFormat Enforcement

Added explicit braces to avoid ambiguous "else" and make GCC happy.

Pragma:no-cache has effect only when Cache-Control is absent.

HTTP/1.1: honour Cache-Control before Pragma:no-cache

HTTPbis WG clarifications on the interaction between Cache-Control and
Pragma specify that Cache-Control may be explicitly crafted to allow
conditional caching by HTTP/1.1 caches and Pragma:no-cache supplied to
prevent caching by older HTTP/1.0 caches unable to understand the
Cache-Control properly.

Squid does obey Cache-Control. So we can deprecate Pragma to the side
cases when Cache-Control is absent entirely.

Reverted trunk r12255 changes. Provided a portable flexible arrays replacement.

Trunk r12255 made Clang compiler happy by removing flexible nonPod[] arrays.
Unfortunately, it also moved shared memory items into local memory (in some
cases uninitialized).

This change provides a Clang-friendly flexible array replacement while keeping
items in the shared memory (and using placement-new initialization). The code
may have become even less readable, but hopefully more portable.

N.B. Flexible arrays were introdiced in C99 standard, after C++ was
standardized in 1998. They are not yet in any revised C++ standard, but most
C++ compilers support them, at least for PODs.

Allow a ufs cache_dir entry to coexist with a shared memory cache entry
instead of being released when it becomes idle.

The original boolean version of the StoreController::dereference() code
(r11730) was written to make sure that idle unlocked local store_table entries
are released if nobody needs them (to avoid creating inconsistencies with
shared caches that could be modified in a different process).

Then, in r11786, we realized that the original code was destroying non-shared
memory cache entries if there were no cache_dirs to vote for keeping them in
store_table. I fixed that by changing the StoreController::dereference() logic
from "remove if nobody needs it" to "remove if somebody objects to keeping
it". That solved the problem at hand, but prohibited an entry to exist in
a non-shared cache_dir and in a shared memory cache at the same time.

We now go back to the original "remove if nobody needs it" design but also
give non-shared memory cache a vote so that it can protect idle but suitable
for memory cache entries from being released if there are no cache_dirs to
vote for them.

This is a second revision of the fix. The first one (r12231) was reverted
because it did not pass tests/testRock unit tests on some platforms. The unit
tests assume that the entry slot is not locked after the entry is stored, but
the first revision of the fix allowed idle entries to remain in store_table
and, hence, their slots were locked and could not be replaced, causing
assertions.  This revision allows the idle entry to be destroyed (and its slot
unlocked) if [non-shared] memory caching is disabled.

It is not clear why only some of the platforms were affected by this. Should
not memory caching be disabled everywhere during testRock (because testRock
does not set memory cache capacity and memory cache entry size limits)?

Fix linking issue in testDiskIO when optimizations are disabled

Windows: filter the buildable helpers for MinGW

libsspwin32: SSP_LogonUser takes char * parameters

There is no need to cast these parameters via void*. Both the library
and the basic auth helper(s) that use it work with char* strings.

Windows: C++ conversion for libsspwin32

Updates this Squid convenience library to build as C++.

Also, fixed header include path for libntlmauth symbol definitions.

Portability: pass OS name to helper config.test scripts

This will help us perform OS-specific enable/disable rather than having
to identify some software or file that SHOULD exist only in those OS, but
which might exist elsewhere for weird reasons (ie windows.h).

NP: the $@ which was being passed was unused by any current config.test
  Removed it and converted to fixed-position parameters for easier
  script development.

Windows: fix error: variable 'h_error' set but not used

Optimization: Test faster condition first. No functionality changes expected.

Bug 3655: pinning failure breaks NTLM and Negotiate authentication

Correct --enable-build-info on out-of-tree builds

bzr expects to be run from the source tree, not the build directory

Added missing library in testStatHist

Windows: better wrap C++-specific code in compat layer

Windows: build Windows specific libraries only on Windows.

Some libraries are only available on Windows and others are unable to be built.

Static Code Analysis: varargs cleanup, improvements to readability

Static Code Analysis: fix varargs handling in snmp_core.cc

Static Code Analysis: fix varargs handling in snmp_core.cc

ssl_crtd: set new certificate serial number to zero

Initialize new BN value to zero instead of simply testing and ignoring
the result.

Detected by Jenkins running clang on FreeBSD 9.

Fix using assignment as a condition without parentheses

SourceFormat Enforcement

Portability: fix several class/struct type mismatches

Leftover problem from trunk r12339 changes.

Detected by clang on FreeBSD 9.

Windows: portability layer updates for MinGW environment

* Move the util.h definitions into os/mswindows.h compat header.
* Add wrapper definitions for many MinGW specific POSIX symbols
* UPdate POSIX wrapper types

A few of these changes are suspect as to whether they work for CygWin and
MSCV. Particularly the POSIX wrapper type changes. We still need a set of
comprehensive MinGW, Cygwin and MSCV build tests to be done. These changes
were enough to build on MinGW some time back, but additional changes are
now likely to be needed and no run-time testing has been done so far.

Fix ipv6 enabled squidclient. Was failing with assertion failure due to ipv6 support not being probed automatically.

FreeBSD: net/if_dl.h seems to require param.h

SourceFormat Enforcement

Portability: typo in EUI header requirement checks

Fix includes missed in rev12369

Windows: simplify abstraction layer header files

Merge compat/os/windows.h, compat/os/mswin.h, and include/squid_windows.h
into a single compat/os/mswindows.h pulled in via the compat.h.

Also, rename compat/mswin.cc to match the new header .h name.

Windows: drop unused WIN32_pipe() implementation

The WIN32_pipe() wrapper function is not necessary on MinGW where _pipe()
is available, and is unused on other Windows builds where helper IPC
layer has been written to use STREAM sockets instead of pipe()

Windows: merge alternative getrusage() into squid_getrusage() wrapper

Since we are already providing a wrapper layer for getrusage() we can
merge the Windows alternative implementation into our wrapper directly.
No need to perform multiple layers of abstraction.

Portability: change Ipc::Port::listen to doListen()

listen() clashes with POSIX function of the same name.
Some systems (ie MinGW) define it as a macro wrapper.

Windows: disable purge tool

The purge tool requires quite a few updates before it will build on Windows.
It is easier at this point to simply not build it.

Windows: wrap sys/shm.h sys/ipc.h includes

Windows: correct C++ type usage on LoadSecurityDll()

ntlm_sspi_auth: correct linkage to library libntlmauth.la

Windows: updated test for ws2tcpip.h and windows.h

Windows: check recv() parameter type

Portability: permit eCAP build without pkg-config

Windows does not have a port of pkg-config, other OS may be in the same
situation. That only impacts on Squids ability to confirm the library
version not on its ability to link and use it.

This reduces some configure errors to warnings when pkg-config is not
located on the local system.

Windows: provide mkdir() wrapper

Windows: update several __MINGW32__ to generic _SQUID_MINGW_

Windows: cleanup Squid OS detection macros

* _SQUID_WINDOWS_  for any Windows build specific code
* _SQUID_CYGWIN_   for CygWin Windows build specific code
* _SQUID_MINGW_    for MinGW Windows build specific code

Windows: fix detection of GNU atomis operations

On MinGW the GNU atomis operatiosn symbols are defined but do not work.
We require a run-time test to detect problems. Not just a compiler check.

Windows: support customization of --prefix= via compat/autoconf.h

Rather than defining DEFAULT_PREFIX on the compiler command line this
builds its value into autoconf.h for use whenever needed. Avoiding
otential issues with buggy Makefile.am coding.

Also allows the --prefix value to be sourced from the autoconf internal
default settings instead of requiring a manual --prefix parameter for
SNMP module to build and link properly.

Windows: disable Mmapped DiskIO support

MinGW does not always supply mmap.

TODO:
 Add proper feature detection to decide when this IS available on Windows.
 Use the Windows native mmap functionality.

Typo in rev12352

Fix error: 'opt_parse_cfg_only' was not declared in this scope

Release Notes: 3.3 updates

Polish: use DBG_PARSE_NOTE to warn about deprecated logformat tokens

Branch 3.3.0.0

Prep for 3.2.2

SourceFormat Enforcement

Do not check 'reply content_length is too big' condition twice.

The check got duplicated was

4410
committer: hno
branch nick: HEAD
timestamp: Wed 2000-05-03 00:49:26 +0000
message:
  hno squid-2.2.STABLE2.to_large_objects-2.patch
  Squid-2.2.STABLE2: Don't swap out objects > maximum_object_size

Added missing source files in recipe for testStatHist

Move fatal family of functions to own implementation and stub files.

Merged from trunk

Shuffled fatal family of functions into own implementation.
Created stub for fatal.cc.
Move all files to src/ due to their dependencies.