#157 dkim_mail_parse does one address only

Ale Vesely

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.


  • Ale Vesely
    Ale Vesely

    Patch from 2.7.2 to libopendkim/dkim-mailparse.new

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

  • Ale Vesely
    Ale Vesely

    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.

    • status: open --> pending
  • 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?

  • 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.

    • status: pending --> open
    • assigned_to: Murray S. Kucherawy
  • Since this is now also on the table for DKIM-Delegate as an experiment, it's done for the next release.

    • status: open --> closed
  • 2.10.0 released.