Re: Rewritten URIBL plugin

2010-07-29 Thread Robert Spier
So if you're interested in the parsed mime functionality, your plugin can plugin_use util/parsed-mime and the right magic happens. Oh yeah that's right, someone *did* implement what you're talking about. You can do it with 'plugin inheritance' (which ironically i knew nothing about until

Re: Rewritten URIBL plugin

2010-07-29 Thread Matt Sergeant
Jared Johnson wrote: sub parse_mime { That works, only this should be called parsed_mime because you're asking for the parsed bit, not telling it to parse (every time). Matt.

Re: per-recipient configuration

2010-07-29 Thread Jared Johnson
I have stuff that might wind up looking like $rcpt-persistent-{statistics_receivedmsgs_total}++; $rcpt-persistent-{statistics_receivedstatsbysender}{$NormalizedSenderAddress}++; Perhaps $rcpt-storage could be provided that could be tied... it just seems like this is the less

Re: per-recipient configuration

2010-07-29 Thread Jared Johnson
colleagues have also complained that notes() even exists as a method and think it would be fine for it to just be a reserved namespace accessed directly as a hash key. or at least that notes() should return the {_notes} hashref if it doesn't have any callers -Jared

Re: per-recipient configuration

2010-07-29 Thread David Nicol
On Thu, Jul 29, 2010 at 11:48 AM, Jared Johnson jjohn...@efolder.net wrote: Nothing's exactly stopping you from Another thing that nothing stops us from, in a late-bound programming language without private namespaces, is adding additional methods to base objects without changing the code that

Re: per-recipient configuration

2010-07-29 Thread Jared Johnson
Another thing that nothing stops us from, in a late-bound programming language without private namespaces, is adding additional methods to base objects without changing the code that declares the base objects, as long as the implementation of the base objects promises to remain the same. I

URIBL plugin 'action' defaults

2010-07-29 Thread Jared Johnson
Would anyone object to setting the default action to 'deny' on certain reliable low-fp URIBL lists? Probably Spamhaus SBL-XBL and DBL and URIBL Black. It seems like a new user turning on uribl checks would expect them to do something more than adding headers, as long as the services it rejects

Re: URIBL plugin 'action' defaults

2010-07-29 Thread Robert Spier
How about instead of hardcoding this, make deny be the default in the sample config? -R Jared Johnson wrote: Would anyone object to setting the default action to 'deny' on certain reliable low-fp URIBL lists? Probably Spamhaus SBL-XBL and DBL and URIBL Black. It seems like a new user

Re: URIBL plugin 'action' defaults

2010-07-29 Thread Robert Spier
And by that, I mean.. I'm not sure I *object*, because there *are* blacklists we trust... but in general, I think we try and avoid doing things that could cause people to lose mail unintentionally. So I think I lean slightly towards the consistent defaults of header tagging side. -R Robert