lp:~measurement-factory/squid/update-right-headers-on-304

Branch merges

Propose for merging

No branches dependent on this one.

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

14722. By eduard-bagdasaryan on 2016-07-06

Merged from trunk r14734

14721. By eduard-bagdasaryan on 2016-06-21

Allow Squid to refresh all headers for 304 responses.

According to RFC 7234 Section 4.3.3, all non-Warning headers
should be updated.
Note that r14582 partially addressed this requirements.

14720. By eduard-bagdasaryan on 2016-06-21

Making Squid conform to HTTPbis refreshing upon
validation requirements.

This is the first change, focusing on "Warning" headers updating.

14719. By Amos Jeffries on 2016-06-20

Bug 4532: pid_filename not working as documented

14718. By Source Maintenance <email address hidden> on 2016-06-17

SourceFormat Enforcement

14717. By Christos Tsantilas on 2016-06-17

Assertion failed: Write.cc:38: "fd_table[conn->fd].flags.open"

The Ftp::Server::stopWaitingForOrigin() notification may come after
Ftp::Server (or an external force) has started closing the control
connection but before the Ftp::Server job became unreachable for
notifications. Writing a response in this state leads to assertions.

Other, currently unknown paths may lead to the same write-after-close
problems. This change protects all asynchronous notification methods
(except the connection closure handler itself) from exposing underlying
code to a closing control connection. This is very similar to checking
for ERR_CLOSING in Comm handlers.

This is a Measurement Factory project.

14716. By Amos Jeffries on 2016-06-17

Remove impossible condition from rev.14715

14715. By Amos Jeffries on 2016-06-17

Unit test for SBuf::reserve()

14714. By Alex Rousskov on 2016-06-15

Do not make bogus recvmsg(2) calls when closing UDS sockets.

comm_empty_os_read_buffers() assumes that all non-blocking
FD_READ_METHODs can read into an opaque buffer filled with random
characters. That assumption is wrong for UDS sockets that require an
initialized msghdr structure. Feeding random data to recvmsg(2) leads to
confusing errors, at best. Squid does not log those errors, but they
are visible in, for example, strace:

  recvmsg(17, 0x7fffbb, MSG_DONTWAIT) = -1 EMSGSIZE (Message too long)

comm_empty_os_read_buffers() is meant to prevent TCP RST packets. The
function now ignores UDS sockets that are not used for TCP.

TODO: Useless reads may also exist for UDP and some TCP sockets.

14713. By Amos Jeffries on 2016-06-15

Bug 4504: Too many WARNING: Ignoring error setting CA certificate locations