[PATCH 1/2] storage: Add "filtered" option, to store based on the filter's decision

Julien ÉLIE julien at trigofacile.com
Sun Jan 21 08:13:49 UTC 2024


Hi Christoph,

> @@ -643,6 +653,7 @@ SMgetsub(const ARTHANDLE article)
>               && (!sub->maxsize || (article.len <= sub->maxsize))
>               && (!sub->minexpire || article.expires >= sub->minexpire)
>               && (!sub->maxexpire || (article.expires <= sub->maxexpire))
> +             && (sub->filtered == article.filtered)
>               && MatchGroups(article.groups, article.groupslen, sub->pattern,
>                              sub->exactmatch)) {
>               if (InitMethod(typetoindex[sub->type]))

When dontrejectfiltered is set in inn.conf, I am under the impression 
that the above additional condition will break storage for people who do 
not use the filtered parameter.
If I do not have the new filtered parameter set on any storage class, 
then filtered articles will no longer be stored ("no matching entry in 
storage.conf").

I am unsure how to best deal with that.
If no storage class has a filtered parameter, fall back to discard that 
condition?

Or consider that using dontrejectfiltered implies the use of the 
filtered parameter in storage.conf, and then it would be a change to 
delay for a major release (INN 2.8.0).  Yet, I would prefer not to 
correlate the two settings, and allow news admins to use 
dontrejectfiltered without having to fiddle with the filtered parameter.

-- 
Julien ÉLIE

« Si bene commemini, causae sunt quinque bibendi : Hospitis aduentus,
   praesens sitis atque futura, Et uini bonitas, et quaelibet altera
   causa. »


More information about the inn-workers mailing list