[Bug 8405] New: idn_to_ascii: conversion to ACE failed - log messages tryinig to convert img src=data: tags

[bu...@spamassassin.apache.org]

https://bz.apache.org/SpamAssassin/show_bug.cgi?id=8405

            Bug ID: 8405
           Summary: idn_to_ascii: conversion to ACE failed - log messages
                    tryinig to convert img src=data: tags
           Product: Spamassassin
           Version: 4.0.2
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: major
          Priority: P2
         Component: spamc/spamd
          Assignee: dev@spamassassin.apache.org
          Reporter: shaun@albino-rabbit.com
  Target Milestone: Undefined

Created attachment 6070
  --> https://bz.apache.org/SpamAssassin/attachment.cgi?id=6070&action=edit
redacted message example

This seems to be related to Bug ID 8302, but in my tests the resolution for
8302 does not solve this.

Identified that messages that contain embedded image data such as:

<img alt=""
src="data:imagepngbase64,iVBORw0KGgoAAAANSUhEUgAAAbQAAACDCAYAAADlPO7n ...

results in bloated noisy mail logs when run as a production mail server 'spamd'
daemon.

My thinking may be naive, but I think a solution would lie in modifying
PerMsgStatus.pm sub 'add_uri_detail_list - in the loop:

foreach my $cleanuri (@uris) {
...
}

add a test to verify that the URI is of a type that supports a remote domain
payload - and if not, skip processing the URI.

e.g.:

next if (substr($cleanuri, 0, 5) eq 'data:');

(sorry, this is a poor example but hopefully it gets the idea across).

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 8405] idn_to_ascii: conversion to ACE failed - log messages tryinig to convert img src=data: tags

[bu...@spamassassin.apache.org]

https://bz.apache.org/SpamAssassin/show_bug.cgi?id=8405

Giovanni Bechis <gi...@paclan.it> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED
                 CC|                            |giovanni@paclan.it

--- Comment #1 from Giovanni Bechis <gi...@paclan.it> ---
Sending        lib/Mail/SpamAssassin/RegistryBoundaries.pm
Sending        t/uri.t
Transmitting file data ..done
Committing transaction...
Committed revision 1936954.

-- 
You are receiving this mail because:
You are the assignee for the bug.