<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, Dec 18, 2016 at 1:22 PM, Juri Haberland <span dir="ltr"><<a href="mailto:juri@sapienti-sat.org" target="_blank">juri@sapienti-sat.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hey Murray,<br>
<br>
good to see you working again on OpenDMARC. Much appreciated!<br>
Hope you will enjoy your vacations and "recharge your batteries" ;)<br>
<br>
Some comments, though:<br>
<span class="gmail-"><br>
On 18.12.2016 11:12, Murray S. Kucherawy wrote:<br>
>       LIBOPENDMARC: Fix bug #165: Fix logic in checking which SPF<br>
>               identifier was used.  Patch from Marco Favero.<br>
<br>
</span>The way you adapted Marco's patch missed one fix, please look at ticket 165<br>
again - I posted a patch that adds the missing bit<br>
(ticket165_incomplete.patch).<br>
<br>
Furthermore I'm missing the following patches, which are running here in<br>
production for several month now without problems and at least two of them<br>
are critical (actually, without #187 beta0/1 can't be compiled, and without<br>
#185 it can't be run without segfaulting).<br>
<br>
#95   SPF validation fail for ipv6<br>
#166  wrong report start and end times in DMARC reports<br>
#185  Segfaults every connection attempts<br>
#187  compile and functional fix for SPF result logging</blockquote><div><br></div><div>Juri:</div><div><br></div><div>I swapped some FB msgs with MSK this morning, and he'll likely be offline until after the holidays.</div><div><br></div><div>I've updated my patch status spreadsheet here:</div><div><br></div><div><a href="https://docs.google.com/spreadsheets/d/1bhlYp_A6j_NF6I9cEAoHsYJV06ooEjkDlsMo9v67mtQ/edit?usp=sharing">https://docs.google.com/spreadsheets/d/1bhlYp_A6j_NF6I9cEAoHsYJV06ooEjkDlsMo9v67mtQ/edit?usp=sharing</a></div><div><br></div><div>I show the following are still missing from Beta1, which one could argue should be included:</div><div><br></div><div>#95</div><div>#153 (MSK says will follow up in a later version)</div><div>#159</div><div>#165_incomplete</div><div>#166</div><div>#179</div><div>#185</div><div>#187</div><div>#193</div><div><br></div><div>I've got it compiling on Fedora with all the above patches except for #159, #179, and #193. Including any of those causes automake problems on Fedora 25.</div><div><br></div><div>I think a patch for #159 and #179 should be combined, since they touch the same file.</div><div><br></div><div>SteveJ</div></div></div></div>