Skip to content

Commit

Permalink
Fix statistics retention default value.
Browse files Browse the repository at this point in the history
60s * 60m * 24h * 31d * 12m * 5 != 5 years
60s * 60m * 365d * 5 != 5 years

But the second value is much closer then using 12 months with
31 days of which 4 months have 30 days and one has either 28
e.g. 3 days less or at best 29 days so 2 days less. When multiplied
with 5 years this mean you calculate something 30+ days to much.
  • Loading branch information
Marco van Wieringen committed Feb 17, 2015
1 parent 48213b2 commit c591fcd
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/dird/dird_conf.c
Expand Up @@ -138,7 +138,7 @@ static RES_ITEM dir_items[] = {
{ "tlskey", store_dir, ITEM(res_dir.tls_keyfile), 0, 0, NULL },
{ "tlsdhfile", store_dir, ITEM(res_dir.tls_dhfile), 0, 0, NULL },
{ "tlsallowedcn", store_alist_str, ITEM(res_dir.tls_allowed_cns), 0, 0, NULL },
{ "statisticsretention", store_time, ITEM(res_dir.stats_retention), 0, ITEM_DEFAULT, "160704000" /* 5 years */ },
{ "statisticsretention", store_time, ITEM(res_dir.stats_retention), 0, ITEM_DEFAULT, "157680000" /* 5 years */ },
{ "verid", store_str, ITEM(res_dir.verid), 0, 0, NULL },
{ "optimizeforsize", store_bool, ITEM(res_dir.optimize_for_size), 0, ITEM_DEFAULT, "true" },
{ "optimizeforspeed", store_bool, ITEM(res_dir.optimize_for_speed), 0, ITEM_DEFAULT, "false" },
Expand Down

0 comments on commit c591fcd

Please sign in to comment.