Allow logging of Postfix queue_id

[Wolfgang Karall-Ahlborn]

Hello,

it would be nice (so this is clearly a wishlist bug) if there was a way to make policyd-spf add the Postfix queue_id to the log lines as other policyd's and milters do.

For example, when doing log analysis, it's difficult to match the python-spf line to the others, here's an excerpt:

policyd-spf[31728]: prepend Authentication-Results: host.example.org; spf=pass (sender SPF authorized) smtp.mailfrom=gmail.com (client-ip=209.85.208.42; helo=mail-ed1-f42.google.com; <email address hidden>; receiver=<UNKNOWN>)
postfix/smtpd[27540]: 5E4543EF9: client=mail-ed1-f42.google.com[209.85.208.42]
postfix/cleanup[32423]: 5E4543EF9: message-id=<email address hidden>
opendkim[11345]: 5E4543EF9: mail-ed1-f42.google.com [209.85.208.42] not internal
opendkim[11345]: 5E4543EF9: not authenticated
opendkim[11345]: 5E4543EF9: DKIM verification successful
opendkim[11345]: 5E4543EF9: s=20210112 d=gmail.com SSL
opendmarc[20059]: 5E4543EF9: gmail.com pass

If policyd-spf would log 5E4543EF9 here, too, it would be easy.

Cheers
Wolfgang

[Scott Kitterman (kitterman)]

Seems reasonable to have this for the policy server since it's only used by postfix.

[Scott Kitterman (kitterman)]

This will only work if you're using the policy post-queue (since, by definition there's no Queue ID before). It'll be included in v3.0.0.

[Wolfgang Karall-Ahlborn (biz+launchpad)]

Hi Scott,

thanks for the change in https://git.launchpad.net/spf-engine/commit/?id=1f1d7c67b8eed13c50866efa39509d514b2dce80

When I applied the patch on some test machine I noticed it is 'QueueID' in policyd_spf.py but 'Queue_ID' in policydspfsupp.py, I had to change the latter also to 'QueueID' to make it work.