~rousskov/squid/3p0-plus

From Revision 8959 to 8940

expand all

collapse all

Rev		Summary	Authors	Date	Diff	Files
8959		Moved comm_local_port TODO comment from pconn reuse code to ... Moved comm_local_port TODO comment from pconn reuse code to comm_open() call: When we reuse a pconn, the F->local_port is already available if it was computed before.	Alex Rousskov	13 years ago
8958		Added %<lp logformat code to record the local port number of... Added %<lp logformat code to record the local port number of the last HTTP server or peer connection.	Alex Rousskov	13 years ago
8957		Synchronized diverged copies of the branch. Synchronized diverged copies of the branch.	Alex Rousskov	13 years ago
8956		Updating initial icap_oldest_service_failure implementation ... Updating initial icap_oldest_service_failure implementation (r8951) to match official Squid code which uses enhanced icap_service_failure_limit. Also, r8951 lacked FadingCounter.{cc,h} files and related bootstrapping changes. Correctly adding FadingCounter.{cc,h} files is difficult because of the test case dependencies. We now include the contents of those files directly into ICAP files that use it.	Alex Rousskov	13 years ago
8955		Fixed store_client.cc:430: "STORE_DISK_CLIENT == getType()" ... Fixed store_client.cc:430: "STORE_DISK_CLIENT == getType()" assertion Merged from v3.0 r9140.	Alex Rousskov	13 years ago
8954		Author: Marko <mr_4u2@yahoo.com> Author: Marko <mr_4u2@yahoo.com> Bug 2496: Downloading some variants in full before relaying AKA, assertion failed: comm.cc:115: "ccb->active == false" if the client disconnected before download finished arriving. Merged from parent (v3.0 r9158)	Alex Rousskov	13 years ago
8953		Fixed header accounting to avoid the "Headers[id].stat.alive... Fixed header accounting to avoid the "Headers[id].stat.aliveCount" assertion.    We were incrementing the alive header field counter twice for each decrement, which probably resulted in the alive counter wrapping back to zero, triggering the assertion. Merged from 3p1-rock r9602.	Alex Rousskov	13 years ago
8952		Fixed comm.cc:152: "ccb->active()" assertion related to the ... Fixed comm.cc:152: "ccb->active()" assertion related to the automatic write timeout. We need to manually cancel writer's interest in select(2) before calling back so that select() does not try to call the write handler when there is no active callback anymore. Normally, select() cancels writer's interest when calling the write handler, but in this case the call back is triggered not from select() but from checkTimeouts(). Ported from 3p1-rock r9597.	Alex Rousskov	13 years ago
8951		Added icap_oldest_service_failure option to forget old ICAP ... Added icap_oldest_service_failure option to forget old ICAP errors. A busy or remote ICAP server may produce a steady but shallow stream of errors. Any ICAP server may become nearly unusable in a short period of time, producing a burst of errors. To avoid disabling a generally usable service, it is important to distinguish these two cases. Just counting the number of errors and suspending the service after icap_service_failure_limit is reached often either suspends the service in both cases or never suspends it at all, depending on the option value. One way to distinguish a large burst of errors from a steady but shallow error stream is to forget about old errors. The added icap_oldest_service_failure option instructs Squid to ignore errors that are "too old" to be counted as a part of a burst. Another way to look at this feature is to say that the combination of the old icap_service_failure_limit and the new icap_oldest_service_failure limits the ICAP error _rate_. For example,    # suspend service usage after 10 failures in 5 seconds:    icap_service_failure_limit 10    icap_oldest_service_failure 5 seconds Squid does not remember every transaction error that occurred within the allowed "oldest error" time period. That would be result in a precise but too expensive implementation, especially during error bursts on a busy server. Instead, Squid divides the period in ten slots, counts the number of errors that occurred in each slot, and forget the oldest slot(s) as needed. Thus, the algorithm has about 90% precision as far as timing of the failures is concerned. That 90% precision ought to be good enough for any deployment.	Alex Rousskov	14 years ago
8950		Revised ClientSocketContext::abort API to handle propagation... Revised ClientSocketContext::abort API to handle propagation of both abort and timeout conditions. With this change, Comm's ETIMEDOUT error reaches http->al.http.timedout. TODO: The server side would benefit from similar timeout/abort checks but it may be difficult to propagate them to al.http from there and most server-side errors are probably already logged as such.	Alex Rousskov	14 years ago
8949		Added write_timeout option to squid.conf. Added write_timeout option to squid.conf. Sometimes, a client gets stuck or disappears when reading the response. Without a write timeout, Squid could wait for hours(!) to terminate the associated transaction.  To add insult to the injury, Squid would log the transaction as TCP_MISS/200 or some other successful outcome because that is how the transaction started (i.e., it was a miss and Squid sent status code 200). Thus, one may get many successful-looking transactions that lasted for many hours while transferring only a few KB of data total. The new write timeout is implemented at Comm level and will apply to all scheduled write() requests. Real-world experience is required to check whether 15 minutes is a reasonable default timeout for each write() event. More work may be needed to propagate all new write timeouts to the access.log.	Alex Rousskov	14 years ago
8948		Append _TIMEDOUT or _ABORTED tags to status access.log field... Append _TIMEDOUT or _ABORTED tags to status access.log field  if the master transaction was terminated because of a timeout or some  other abnormal event like an I/O error. Without this change, it is often impossible to distinguish a failed transaction from a successful one, especially when the failure comes after the we sent the status code to the client. These extra tags allow log processing scripts to treat marked transactions specially. For example, timedout transaction response time does not mean much and should be excluded from most response time statistics.	Alex Rousskov	14 years ago
8947		Break forwarding loops when working as a transparent proxy. Break forwarding loops when working as a transparent proxy. Squid detects forwarding loops in most configurations, but breaks them (using a customizable HTTP_FORBIDDEN response) only when working as an accelerator. With this change, Squid now breaks loops when working as a transparent proxy as well. Interestingly enough, the old breaking code comment says that both cases are covered. Perhaps the exclusion of transparent mode was not done on purpose.	Alex Rousskov	14 years ago
8946		Truncate too-long HTTP response bodies to match their Conten... Truncate too-long HTTP response bodies to match their Content-Length header. Sometimes a broken server sends more than Content-Length bytes in the response.  For example, a 302 redirect message with "Content-Length: 0" header may include an HTML body. Squid used to send "everything" it read to the client, even if it read more than the Content-Length bytes. That may have helped in some cases, but I think we should be more conservative when dealing with broken servers to combat message smuggling attacks and other bad side-effects for clients.  We now do not forward more than the advertised content length and declare the connection with a broken server non-persistent. Chunked responses (that Squid should not receive and that must not have a Content-Length header) are not truncated because RFC 2616 says we MUST ignore their Content-Length header. TODO: simply truncating read content would not work for pipelined responses. We should preserve extra content for the next transaction on a pconn.	Alex Rousskov	14 years ago
8945		Converted strange assert(strstr(url, request->urlpath.buf())... Converted strange assert(strstr(url, request->urlpath.buf())) assertion into a warning until we understand what the assertion tests and why it tests that where it does.	Alex Rousskov	14 years ago
8944		When appending to a String, do not allow the String length d... When appending to a String, do not allow the String length data member to overflow, leading to segfaults when we start copying into a smaller-than-we-think String buffer. This is a stop-gap measure to convert segfaults into assertions. The true fix is to either make the calling code check before appending or, better, throw an exception that the call chain can handle without bringing the whole Squid down.	Alex Rousskov	14 years ago
8943		Polished version string to match Launchpad URL. Polished version string to match Launchpad URL.	Alex Rousskov	14 years ago
8942		Aggressively prevent X-Forwarded-For growth during infinite ... Aggressively prevent X-Forwarded-For growth during infinite forwarding loops. To make growth-associated problems visible, not only Via support has to be turned off, but request_header_max_size has to be raised or disabled as well.	Alex Rousskov	14 years ago
8941		Prevent infinite X-Forwarded-For growth during infinite forw... Prevent infinite X-Forwarded-For growth during infinite forwarding loops. X-Forwarded-For growth leads to String::initBuf size limit assertions and probably other problems. Infinite forwarding loops may happen if forwarding loop detection is disabled with "via off" but forwarding loops are happening. More work is needed to abort looping transactions instead of limiting X-Forwarded-For growth.	Alex Rousskov	14 years ago
8940		Do not dereference nil client_address when debugging at leve... Do not dereference nil client_address when debugging at level 6+.	Alex Rousskov	14 years ago

• Older »

Loggerhead is a web-based interface for Breezy
Version: 2.0.1