[rt-users] Improper display of ticket transaction's HTML

[Jeff Blaine]

An update, below, on this issue in case anyone comes across it. Seems
like a bug to me.

On 3/19/2015 9:16 AM, Jeff Blaine wrote:
> On 3/11/2015 3:56 AM, Christian Loos wrote:
>> Hi,
>>
>> upgrading to a more recent version of RT and installing HTML::Gumbo
>> gives you tables in transactions (RT supports this since 4.2.6) [1].
>>
>> If you can't upgrade, you can use RT::Extension::PermissiveHTMLMail [2].
>>
>> Chris
>>
>> [1] https://www.bestpractical.com/docs/rt/4.2/RT_Config.html#PreferRichText
>> [2] https://github.com/bestpractical/rt-extension-permissivehtmlmail
> 
> GOOD:
> 
> We've updated our test server to RT 4.2.10 and installed HTML::Gumbo per
> the notes in the documentation for $PreferRichText. When
> local/lib/RT/Interface/Web_Local.pm is configured to allow various table
> attributes, including 'style', the transaction HTML now displays as a
> table properly!
> 
> BAD:
> 
> But now there are various html, head, body tags on what seems to be only
> search result blocks.
> 
> IIRC, image attachments are not allowed to the list. Here are 2
> screenshots showing what I am talking about in case anyone wants to see:
> 
> https://www.dropbox.com/s/euca82h6qfagki1/rt-4210-broken.jpg?dl=0
> 
> https://www.dropbox.com/s/ym2ctowlo8qhmnf/rt-4210-broken2.jpg?dl=0

This problem goes away if I uninstall HTML::Gumbo

>> Am 10.03.2015 um 20:35 schrieb Jeff Blaine:
>>> Hi,
>>>
>>> We're using RT 4.2.5 (for now).
>>>
>>> $PreferRichText is turned on.
>>>
>>> We've put in place a Web_Local.pm that is allowing all tags via:
>>>
>>>     push @SCRUBBER_ALLOWED_TAGS, qw('*');
>>>
>>> Something is still mucking with our HTML though, specifically (that
>>> we've found so far) various table tags' attributes like borders and
>>> padding seem to not be rendered properly. The table borders don't render
>>> at all.
>>>
>>> Any ideas?
>>>
>>> It is a little unclear to me exactly how to use
>>> %SCRUBBER_ALLOWED_ATTRIBS ... setting it to {'*', 1} caused various
>>> links in the web UI to not work anymore.
>>>
>>
> 

-- 
Jeff Blaine
kickflop.net
PGP/GnuPG Key ID: 0x0C8EDD02