OpenDMARC icon indicating copy to clipboard operation
OpenDMARC copied to clipboard

opendmarc_policy_store_dkim signature change between 1.3 and 1.4

Open grobian opened this issue 4 years ago • 3 comments

In commit dbd87868f2ca9c2ef11529cd757d1cc5ab228833 the signature of opendmarc_policy_store_dkim changed. This breaks with Exim (https://bugs.exim.org/show_bug.cgi?id=2728) and there's no way this can be resolved other than a configure check at Exim to check how many arguments opendmarc_policy_store_dkim needs.

This isn't quite cool.

May I suggest to: a) bump your #define OPENDMARC_LIB_VERSION 0x00000000 to something useful (so an ifdef in the consumer code could do something to fix this) b) change your major revision of your libary next time you break the API c) consider what these kinds of changes break if you simply upgrade opendmarc without recompiling the consumers, we probably get a spectacular crash of some sort (best case) or some very odd behaviour?

grobian avatar May 08 '21 08:05 grobian

This has recently celebrated its first birthday. What are you planning to do about this issue?

Zugschlus avatar May 11 '22 19:05 Zugschlus

Given that Exim is the most used MTA

It stands to reason this bug should be looked at ASAP.

brentclark avatar May 12 '22 11:05 brentclark

Is there anything I missed on this issue? Right now I'm about to introduce a new DMARC_API option for Exim's build config (EDITME). But the cleaner way would be a meaningful OPENDMARC_LIB_VERSION in dmarc.h

HeikoSchlittermann avatar Oct 15 '22 21:10 HeikoSchlittermann