checkincludedtext behavior

[Jeffrey M. Vinocur]

On Tue, 10 Dec 2002, Russ Allbery wrote:

> Jeffrey M Vinocur <jeff at litech.org> writes:
> 
> > Right, Alex's point about removing it entirely crossed my mind as well.
> 
> I'd kind of like to see that happen, just because that way it's very clear
> what our stance is on the thing.  I don't really like having code that
> can't be easily customized for local behavior but that needs such
> customization sitting around in the core code.

Ok, I just commited a quick stab at this (disabled) to the sample 
filter_nnrpd.pl in CURRENT.  Tested mildly, works just fine.  You have to 
edit the script to configure it, but the configuration stuff is at least 
pulled out to the top so that anybody who wants to plug something tricky 
can.

Code is at http://www.litech.org/~jeff/inn-diffs/checkincludedtext for
those who might want it urgently.

I haven't removed the support from nnrpd yet -- Russ, I'll leave that to 
your discretion I think.


> That's a good idea.  nnrpd should really pass the identity assigned by
> readers.conf to the posting filter as an argument.  Maybe something to
> tackle when we clean up the interface to the posting filters.  That's an
> INN 2.5 thing.

Yeah, I don't think I'll do it today or anything.

(Well, the actual identity I believe is visible, as $user.  But what I'd
like to see is the label of the access group being used.)


-- 
Jeffrey M. Vinocur
jeff at litech.org