Improper display of ticket transaction's HTML

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.

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] RT Config - RT 4.2.17 Documentation - Best Practical
[2] https://github.com/bestpractical/rt-extension-permissivehtmlmailAm 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.

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] RT Config - RT 4.2.17 Documentation - Best Practical
[2] GitHub - 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:

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

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

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] RT Config - RT 4.2.17 Documentation - Best Practical
[2] GitHub - 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:

Dropbox - File Deleted - Simplify your life

Dropbox - File Deleted - Simplify your life

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

hi, use this on Web_Local.pm:

$SCRUBBER_ALLOWED_ATTRIBUTES{$_} = 1 for qw/colspan rowspan align valign cellspacing cellpadding border width height style/;