Bug #19063 for HTML-Scrubber: doesn't support generating valid xhtml

Thu May 04 03:56:06 2006 Guest - Ticket created

Subject:

doesn't support generating valid xhtml

I've had a bug reported against the Debian libhtml-scrubber-perl package that I maintain: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=365971 "HTML::Scrubber seems to have no support for generating valid XHTML 1.0 Strict. It converts incoming valid <hr /> to <hr>. Probably the best way to fix this would be to add a config option that turns on using <hr /> and similar."

Fri May 05 17:52:27 2006 Guest - Correspondence added

On Thu May 04 03:56:06 2006, guest wrote: Show quoted text

> I've had a bug reported against the Debian libhtml-scrubber-perl package > that I maintain: > > http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=365971 > > "HTML::Scrubber seems to have no support for generating valid XHTML 1.0 > Strict. It converts incoming valid <hr /> to <hr>. Probably the best way > to fix this would be to add a config option that turns on using <hr /> > and similar."

The slash is treated like an attribute, for example see http://search.cpan.org/src/PODMASTER/HTML-Scrubber-0.08/t/07_booleans.t

Fri May 05 17:52:27 2006 The RT System itself - Status changed from 'new' to 'open'

Wed May 10 23:30:04 2006 Guest - Correspondence added

I can't get this to work use HTML::Scrubber; my @allow = qw[ br hr b a option button th style script]; my @default = ( 1, # this determines wthere xhtnl style accepted { '/' => 0, href => 0, } ); my $scrubber = HTML::Scrubber->new( allow => \@allow, default => \@default ); my $html = q[ <style type="text/css"> BAD { background: #666; color: #666;} </style> <script language="javascript"> alert("Hello, I am EVIL!"); </script> <HR> a => <a href=1>link </a> br => <br/> b => <B onclick="hello()"> bold </B> u => <U> UNDERLINE </U> ]; print $scrubber->scrub($html),"\n"; It seems to be the first parameter to @default that determines whether the tailing slash is recognized or not. The '/' attribute seems to be ignored