[MERGE] Bug 2267 - increase MAX_URL to 8192

2008-04-04 Thread Amos Jeffries
Bug 2267 - Request increase MAX_URL to 8192
# Bazaar merge directive format 2 (Bazaar 0.90)
# revision_id: [EMAIL PROTECTED]
# target_branch: file:///src/squid/bzr/trunk/
# testament_sha1: de0fc5d94f18f06cd0db40ebc3143d26ee4811c3
# timestamp: 2008-04-05 02:00:42 +1200
# base_revision_id: [EMAIL PROTECTED]
#   e2j0pkbbcl3uc4c2
# 
# Begin patch
=== modified file 'src/defines.h'
--- src/defines.h	2008-03-20 11:30:19 +
+++ src/defines.h	2008-04-04 13:59:45 +
@@ -154,7 +154,7 @@
 
 #define MAX_FILES_PER_DIR (120)
 
-#define MAX_URL  4096
+#define MAX_URL  8192
 #define MAX_LOGIN_SZ  128
 
 #define PEER_MAX_ADDRESSES 10

# Begin bundle
IyBCYXphYXIgcmV2aXNpb24gYnVuZGxlIHY0CiMKQlpoOTFBWSZTWYQntzAAAYzfgAAQWWP//3IW
EkCwUAM685VXTR3ZWGSImmmjIbUaZPUDRoAaNAACSKU9N6keSMUPU0A0NBo0AGhoEigVMnqe
p5Rsk9TQABoGj1DQAYAABJIQDRNGETNBT0xEDGkaMJpXZhY8bcS+E5qA1ke7ZSMgJXVR
qlHl17O9aadhtlmpeHHnlIikgpRjLPEQJVD95NWtPKCpAWSYdFuPo2Vy46itQTtOh4jL51FLWFUS
QR0ILdCsCLbUjWtiRUW96G7Mp+4lFS2lIxGszOumTxOQOFjAebwd6ipBQVFdRo+VzzE1Wc0SVGSZ
zSVsPdQPDGUWs+CwKVLV1WXq1SNys5S0wHLckCGkszp3VieeMo04npCkoKLLFULRIMq62kk5jjNd
KEkWAgOucYJHsKUWxVDoGGFhITzK2LF9/xdmvgxj0For5+dIZVj7ltA8m2MYPFiXC4byfHmUJ1At
YRlqLs9Zea1pN2CRSHKNUFjihVUgxhhOIgRWMchZtnE4mRGMSMKhiTPQJ9apHLUMUjTma1Xp9RFK
TA+qoxeLMNyZVDnhAzHW1nxOHRGkCjhG9I+CEa2ma7HZeox7Hwex8FFivcDs3XPIODYcDgbzkUg4
Y88Oew2yTFhtDxzlr4zin+66cSQ8l3LQQDPu3BNHkO6dBjaF19LTWpI6IxXMXzhIoTpnRszdrSym
+e8Z9ZZr8FcPt4iiUcLjzJd9TRgGRE4EcuruTwX+8QrSH9UyH25sNIZXpi9OSJwJU+nsO0zmJWMX
FRN6b+Dwscpu7tMhe/JbDENQpsK02JXUkflD4FxYnGYawFgP1A/GLrRACIATKnzmMbMZugLNYzGW
9LYokPChpzZS3t1U4jApOANLOTlcNEmuRIe2xf0Qg0MuCRlu0pFzSAc3ogEi1uWiM8BehC0wwvUC
cO9T9otk843kLakasFSHOFudI3zCp46Isn1S3xVD7QmSY021oHMbrNq5ZDABzX3LyZYQTeVXyRQ9
IwPDjAl/pYNG8lx0ZEFME6+UhlBlc44r7fpvokYbE4yQFoMgAo2uhZhvJSpTHyRgjD18fxdyRThQ
kIQntzA=


Re: Rev 8911: Honor 0x and 0 prefixes as numeric base indication when parsing squid.conf in http://squid-cache.org/bzr/squid3/trunk/

2008-04-04 Thread Henrik Nordstrom
tor 2008-04-03 klockan 08:24 -0600 skrev Alex Rousskov:

 The following is the message I had to manually forward to the commit
 list because I have not yet learned how to configure bzr to
 automatically pick sendto addresses based on the branch. The trunk
 commit message was sent to me and I forwarded it.

Robert has a cron job which sends mails about commits on trunk, at least
most of the time..

Regards
Henrik



$(SHELL) for test-suite/testheaders.sh

2008-04-04 Thread Alex Rousskov
Amos,

Could you please change Makefile.ams so that
test-suite/testheaders.sh is executed using $(SHELL)? For example,

$(SHELL) $(top_srcdir)/test-suite/testheaders.sh ...

There may be a better, automake-assisted way of running shell scripts,
but I do not know it.

Without SHELL or equivalent, if bash is not installed in /bin/, the user
of make check gets an error:


 gmake[2]: Entering directory `/home/rousskov/Edit/squid3/ecap/lib'
 gmake  testHeaders tests/testAll
 gmake[3]: Entering directory `/home/rousskov/Edit/squid3/ecap/lib'
 ../test-suite/testheaders.sh g++ -DHAVE_CONFIG_H -I. -I../include 
 -I../include -I../include -I/usr/local/include   -Werror -Wall 
 -Wpointer-arith -Wwrite-strings -Wcomments  -g  ../include || exit 1
 /usr/local/bin/bash: ../test-suite/testheaders.sh: /usr/bin/bash: bad 
 interpreter: No such file or directory
 gmake[3]: *** [testHeaders] Error 1

It would be nice to have the name of the testheaders.sh script as a
Makefile.am variable (perhaps even stored in one common.am file which is
included by all Makefiles), but that can wait.

Thank you,

Alex.





offline_mode

2008-04-04 Thread Amos Jeffries
Bug 1356 brings up some confusion over what the true meaning of 
'offline' is for this option.


As Henrik pointed out fairly early in the discussion there, squids 
actual behavior is to 'aggressively cache', not a true 
network-disconnected mode.


I'm proposing that the option have one or both of two things done to it:

1) simply renamed to something more descriptive ie cache_aggressive on/off.

2) that it be kept as 'offline' but changed to accept a range of flags 
indicating which of the external-interface engines inside squid are to 
behave as if they were offline. ie offine cache dns auth


When any given feature is marked for offline it behaves in a true not 
packet leaves the interface fashion.

ie
- 'dns' means no new DNS lookups are performed.
	- 'cache' means no refreshes are performed (RFC 2616 allows for stale 
objects already in cache to be used with certain response types)

- etc.

We may need to exempt localhost traffic from offline mode explicitly. 
But will need to be consistent with it.


The existing 'aggressive caching' behaviour may be possible by other 
means with cache ACL or refresh_patterns. Or a 'cache=semi' style tag 
might be chosen.


Or, we may choose to do both of the above changes.

Amos
--
Please use Squid 2.6.STABLE19 or 3.0.STABLE4