Bug #55629 for HTML-Parser: Wrong parse

Tue Mar 16 11:09:51 2010 NIKOLAS [...] cpan.org - Ticket created

Subject:

Wrong parse

HTML: <iframe/**/src="http://mail.ru" name="poc iframe jacking" width="100%" height="100%" scrolling="auto" frameborder="no"></iframe> $parser = HTML::Parser->new( api_version => 3, start_h => [ sub{ my ($Self, $Text, $Tag, $Attr) = @_; print "Tag is: ".$Tag; }, "self, text, tagname, attr" ] ); $parser->ignore_elements( qw( iframe )); $parser->ignore_tags( qw( iframe )); output: Tag is: iframe/**/src="http://mail.ru"

Thu Mar 18 09:51:31 2010 NIKOLAS [...] cpan.org - Correspondence added

Втр Мар 16 11:09:51 2010, NIKOLAS писал: Show quoted text

> HTML: > <iframe/**/src="http://mail.ru" name="poc iframe jacking" width="100%" > height="100%" scrolling="auto" frameborder="no"></iframe> > > $parser = HTML::Parser->new( > api_version => 3, > start_h => [ sub{ > my ($Self, $Text, $Tag, $Attr) = @_; > print "Tag is: ".$Tag; > }, "self, text, tagname, attr" ] > ); > $parser->ignore_elements( qw( iframe )); > $parser->ignore_tags( qw( iframe )); > > output: > Tag is: iframe/**/src="http://mail.ru"

HTML: <script/src="ya.ru"> wrong parse same

Thu Mar 18 09:51:35 2010 NIKOLAS [...] cpan.org - Status changed from 'new' to 'open'

Sun Apr 04 16:38:08 2010 GAAS [...] cpan.org - Correspondence added

I don't understand what rules you propose that HTML::Parser should follow to parse this kind of bogus HTML. You think it should treat "/**/" and "/" as whitespace?

Tue Jun 01 03:13:54 2010 NIKOLAS [...] cpan.org - Correspondence added

Here 3 regular expressions applied to the entrance text correct this problems: s{(/\*)}{ $1}g; s{(\*/)}{$1 }g; s{(<[^/\s<>]+)/}{$1 /}g; Probably you will find more correct architectural decision.

Mon Aug 24 14:36:46 2020 olaf [...] wundersolutions.com - Correspondence added

Ticket migrated to github as https://github.com/libwww-perl/HTML-Parser/issues/12

Mon Aug 24 14:36:47 2020 olaf [...] wundersolutions.com - Status changed from 'open' to 'resolved'