Bug 10311

Summary: Squid compile option --enable-cache-digests missing
Product: IPFire Reporter: Jan_B <jan.behrensx>
Component: squidAssignee: Michael Tremer <michael.tremer>
Status: CLOSED FIXED QA Contact:
Severity: Minor Usability    
Priority: - Unknown - CC: michael.tremer
Version: 2   
Hardware: all   
OS: All   
Attachments: lfs file for squid
proxy.cgi and languages de&en configured
file: lfs/squid
proxy.cgi
de.pl
en.pl

Description Jan_B 2013-02-24 12:37:09 UTC
Created attachment 116 [details]
lfs file for squid

The option to enable cache-digests is missing.
Attached is a LFS-File for squid with enabled cache-digests.
File-Path: ipfire-2.x / lfs / squid
Comment 1 Michael Tremer 2013-02-24 14:20:04 UTC
In what way is this missing? What are the advantages/disadvantages?
Comment 2 Jan_B 2013-02-24 14:35:53 UTC
In Multi-Squid environments it allows to exchange cache-digests between squid-peers.

The disadvantage is an automatic generation of cache-digests every 10minutes which are hold in memory, that are only some KByte. On small memory servers this could be a disadvantage.

You can disable this behaviour with 
digest_generation off
in squid.conf

This should be taken into the default squid.conf user-includefile to let the user decide to enable it on demand.

More information about the cache-digests is found here:
http://wiki.squid-cache.org/SquidFaq/CacheDigests

There has already been a discussion on forum for this:
http://forum.ipfire.org/index.php?topic=7354.0
Comment 3 Michael Tremer 2013-02-24 18:23:24 UTC
Yeah, I can live with that if we compile in the code and disable it by default in the configuration. People who need it can switch it on.

Could you please provide a patch that enables the compile option and alters the configuration file? The best option would be to have a check box in the web user interface.
Comment 4 Jan_B 2013-02-24 21:34:52 UTC
Created attachment 117 [details]
proxy.cgi and languages de&en configured
Comment 5 Jan_B 2013-02-24 22:14:43 UTC
Created attachment 118 [details]
file: lfs/squid
Comment 6 Jan_B 2013-02-24 22:15:41 UTC
Created attachment 119 [details]
proxy.cgi

please adjust foldernames of patch
Comment 7 Jan_B 2013-02-24 22:16:06 UTC
Created attachment 120 [details]
de.pl

please adjust foldernames of patch
Comment 8 Jan_B 2013-02-24 22:16:20 UTC
Created attachment 121 [details]
en.pl

please adjust foldernames of patch
Comment 9 Jan_B 2013-02-24 22:18:35 UTC
patched some files, please try.
Comment 10 Michael Tremer 2013-03-01 00:05:34 UTC
http://git.ipfire.org/?p=ipfire-2.x.git;a=commitdiff;h=6bea848d3476dd042d0a4f0f9eca63f7773e503c

Merged. Please review the commit and report any problems. This will probably be shipped with core update 67 which is not scheduled, yet.