Mailing List Archive: [Bug 3080] Authentication-Results Header iprev uses old/wrong property for client IP address

https://bugs.exim.org/show_bug.cgi?id=3080

--- Comment #1 from Wolfgang Breyha <wbreyha@gmx.net> ---
Created attachment 1473
--> https://bugs.exim.org/attachment.cgi?id=1473&action=edit
proposed patch

After a short discussion at
https://github.com/lieser/dkim_verifier/issues/436
it looks like RFC 5322 wants us to quote IPv6 addresses in headers as well.

This patch changes to policy.iprev and quotes v6. I've built it on my local
host and it looks good for v4/v6.

I did not check the influence to ARC yet, but if I read the source right it
takes the contents of the existing ARH?

--
You are receiving this mail because:
You are on the CC list for the bug.

--
## subscription configuration (requires account):
## https://lists.exim.org/mailman3/postorius/lists/exim-dev.lists.exim.org/
## unsubscribe (doesn't require an account):
## exim-dev-unsubscribe@lists.exim.org
## Exim details at http://www.exim.org/
## Please use the Wiki with this list - http://wiki.exim.org/

https://bugs.exim.org/show_bug.cgi?id=3080

Wolfgang Breyha <wbreyha@gmx.net> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #1473|0 |1
is obsolete| |

--- Comment #2 from Wolfgang Breyha <wbreyha@gmx.net> ---
Created attachment 1474
--> https://bugs.exim.org/attachment.cgi?id=1474&action=edit
improved patch

new patch. different approach.
adds quoting of IPv6 for ARC smtp.remote-ip as well

--
You are receiving this mail because:
You are on the CC list for the bug.

--
## subscription configuration (requires account):
## https://lists.exim.org/mailman3/postorius/lists/exim-dev.lists.exim.org/
## unsubscribe (doesn't require an account):
## exim-dev-unsubscribe@lists.exim.org
## Exim details at http://www.exim.org/
## Please use the Wiki with this list - http://wiki.exim.org/