On 23.4.2008, Israel Herraiz wrote: > I am using list_address.email to automatically generate labels for > messages that are coming from mailing lists (that field uses the value > of the List-Post header of the raw message). It seems that some > mailing lists are not including that header though. In those lists > where I do not see List-Post, I have found that List-Id is still > present. Therefore I propose to add that header to the message class > for filtering purposes.
Adding list-id is a good idea I think but you cant do a personmanager lookup with it because it isnt really an email address according to the RFC [1]. List-Subscribe and List-Unsubscribe all set their counterparts in message.rb to the raw header so this is what I would expect list_id to get set to. [snip] > @@ -128,6 +128,12 @@ class Message > @source_marked_read = header["status"] == "RO" > @list_subscribe = header["list-subscribe"] > @list_unsubscribe = header["list-unsubscribe"] > + @list_id = > + if header["list-id"] > + @list_id = PersonManager.person_for header["list-id"] > + else > + nil > + end > end > private :parse_header [snip] If you are labeling via list-id you just want the raw header so this should just be @list_id = header["list-id"] Then if you want to do a lookup you can do it in the before-add-message hook. Marcus [1] http://www.apps.ietf.org/rfc/rfc2919.html _______________________________________________ sup-talk mailing list [email protected] http://rubyforge.org/mailman/listinfo/sup-talk
