Because peerGetSomeParent and then peerGetAllParents are both called in the NEVER_DIRECT case (e.g., accelerators), the list of peers to use will contain a duplicate; whatever peerGetSomeParent returns will be in the list twice, and if it's the first peer (lexically in the conf file), it'll be twice in a row. This isn't good, especially for round-robin accelerator setups. Attached patch skips duplicates of the first peer list entry (in the long term, this might need to be reworked, but this seems like a low- cost way to improve things for the time being).
<http://www.squid-cache.org/bugs/show_bug.cgi?id=2378>

--
Mark Nottingham       [EMAIL PROTECTED]


Reply via email to