#157 dkim_mail_parse does one address only

2.7.2
closed
5
2014-12-28
2012-11-29
Ale Vesely
No

It is not practical to use dkim_mail_parse to get multiple addresses, because one would have to break a mailbox-list into single mailboxes and then call dkim_mail_parse on each of them.

I attach my attempt at it. (It also moves the is_spacial table to static storage, which is not directly related.) The patched function is still suboptimal for, say, 10K-sized To: fields, but it seems to work in some cases.

Discussion

  • Ale Vesely

    Ale Vesely - 2012-11-29

    Patch from 2.7.2 to libopendkim/dkim-mailparse.new

     
  • Murray S. Kucherawy

    Is the goal here to check multiple From: values for signing, or something else?

     
  • Ale Vesely

    Ale Vesely - 2012-11-30

    Multiple From: mailboxes will be needed, sooner or later. However, no, I want to read the recipients of the messages I sign, for logging/whitelisting purposes. Given the mailbox syntax, it is quite handy to have such a stdalone parsing function.

     
  • Murray S. Kucherawy

    • status: open --> pending
     
  • Murray S. Kucherawy

    I'm still having trouble understanding the use case here. To which configuration item is this to be connected? You want to reject mail except when addressed to certain users?

     
  • Murray S. Kucherawy

    Revisiting this: I suspect the most useful hook for this will be into the Lua functions, where this could return an array of parsed addresses and you could take some logging action based on that. Is that what you had in mind?

    If not, I'm still confused as to how this would be exposed to the operator.

     
  • Murray S. Kucherawy

    • status: pending --> open
    • assigned_to: Murray S. Kucherawy
     
  • Murray S. Kucherawy

    Since this is now also on the table for DKIM-Delegate as an experiment, it's done for the next release.

     
  • Murray S. Kucherawy

    • status: open --> closed
     
  • Murray S. Kucherawy

    2.10.0 released.

     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks