[Tickets #9567] Re: charset pb replying to message

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/9567
------------------------------------------------------------------------------
  Ticket             | 9567
  Updated By         | Michael Slusarz <slusarz at horde.org>
  Summary            | charset pb replying to message
  Queue              | IMP
  Version            | Git master
  Type               | Bug
  State              | Feedback
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             | Michael Slusarz
------------------------------------------------------------------------------


Michael Slusarz <slusarz at horde.org> (2011-03-15 17:04) wrote:

> --- Xss.php.org	2011-03-15 10:41:22.000000000 +0100
> +++ Xss.php	2011-03-15 10:41:24.000000000 +0100
> @@ -130,7 +130,7 @@
>              }
>          }
>
> -        return Horde_String::convertCharset($text, $dom->encoding,  
> $this->_params['charset']);
> +        return $text;
>      }

This isn't correct.  Xss filter needs to return text in whatever  
charset it was provided in, which is why the convertCharset() call is  
necessary.  The question is why $dom->encoding is 'ISO-8859-1' for you  
and 'UTF-8' for *everybody* else.