https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7215
Mark Martinec <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #5313|0 |1 is obsolete| | --- Comment #3 from Mark Martinec <[email protected]> --- Created attachment 5322 --> https://bz.apache.org/SpamAssassin/attachment.cgi?id=5322&action=edit Introduce Util::idn_to_ascii and make use of it Here is a focused variant of the previously attached patch, which provides the Util::idn_to_ascii (which calls Net::LibIDN if it is available and is needed), and makes use of it where necessary (i.e. wherever DNS query is being assembled). The following feature as mentioned in the opening posting is _not_ included with this patch - it should be implemented elsewhere: | Instead of the more complex task of fixing | the text parser, as a stop-gap solution I added some sanitation | code for extracted URIs: trimming prefix and suffix characters | that cannot appear in a valid Unicode URI. This sanitation code | would eventually be removed when a parser is improved. Similarly the user-defined character classes InIDNAWhitespace, InIDNAFullStop, and InIDNA2008 are _not_ included in this patch so that it can remain clean and focused on a single task of introducing idn_to_ascii() and is_valid_utf_8(). trunk: Sending lib/Mail/SpamAssassin/AsyncLoop.pm Sending lib/Mail/SpamAssassin/Dns.pm Sending lib/Mail/SpamAssassin/DnsResolver.pm Sending lib/Mail/SpamAssassin/Plugin/AskDNS.pm Sending lib/Mail/SpamAssassin/Plugin/HeaderEval.pm Sending lib/Mail/SpamAssassin/Plugin/URIDNSBL.pm Sending lib/Mail/SpamAssassin/Util.pm Committed revision 1694252. Please yell if this is for some reason unacceptable :) -- You are receiving this mail because: You are the assignee for the bug.
