Bug #46099 for HTML-Parser: Make iframe parsing configurable

# Fri May 15 02:15:45 2009 bdfoy [...] cpan.org - Ticket created

Subject:	Make iframe parsing configurable
MIME-Version:	1.0
X-Mailer:	MIME-tools 5.427 (Entity 5.427)
Content-Type:	text/plain
Charset:	utf8
Content-Disposition:	inline
Content-Transfer-Encoding:	binary
X-RT-Original-Encoding:	utf-8
Content-Length:	380

Download (untitled) / with headers
text/plain 380b

Since the latest versions of HMTL::Parser do not parse the content of iframes, some of my applications using HTML::SimpleLinkExtor have broken. The text between the iframe tags is what the browser displays and is usually more HTML, and I need to be able to extract any links in that text. I'd like to at least be able to turn on parsing for iframes, even if it is off by default.

# Sat Jun 20 05:17:40 2009 GAAS [...] cpan.org - Correspondence added

MIME-Version:	1.0
X-Mailer:	MIME-tools 5.427 (Entity 5.427)
Content-Disposition:	inline
Charset:	utf8
Content-Type:	text/plain
Message-ID:	<rt-3.6.HEAD-13950-1245489460-1333.46099-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding:	binary
X-RT-Original-Encoding:	utf-8
Content-Length:	778

Download (untitled) / with headers
text/plain 778b

On Fri May 15 02:15:45 2009, BDFOY wrote: Show quoted text

> Since the latest versions of HMTL::Parser do not parse the content of > iframes, some of my applications using HTML::SimpleLinkExtor have > broken. The text between the iframe tags is what the browser displays > and is usually more HTML, and I need to be able to extract any links in > that text.

Browsers that support iframes are supposed to ignore everything inside the iframe. They are supposed to render the HTML found at the 'src' location. Show quoted text

> I'd like to at least be able to turn on parsing for iframes, even if it > is off by default.

I see the point if you need to emulate the behaviour of very old browsers. A workaround is to invoke a subparser on the iframe content text. I'll see if I find an easier way to do this.

# Sat Jun 20 05:17:41 2009 The RT System itself - Status changed from 'new' to 'open'

# Sat Jun 20 05:24:09 2009 GAAS [...] cpan.org - Correspondence added

MIME-Version:	1.0
X-Mailer:	MIME-tools 5.427 (Entity 5.427)
Content-Disposition:	inline
Charset:	utf8
Content-Type:	text/plain
Message-ID:	<rt-3.6.HEAD-13950-1245489849-324.46099-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding:	binary
X-RT-Original-Encoding:	utf-8
Content-Length:	223

Download (untitled) / with headers
text/plain 223b

The TODO file has this entry: - make literal tags configurable. The current list is hardcoded to be "script", "style", "title", "iframe", "textarea", "xmp", and "plaintext". which would be my preferred way to fix this.

# Tue Sep 20 13:20:09 2011 JWILLIAMS [...] cpan.org - Correspondence added

MIME-Version:	1.0
X-Mailer:	MIME-tools 5.427 (Entity 5.427)
Content-Disposition:	inline
Content-Type:	text/plain; charset="UTF-8"
Message-ID:	<rt-3.8.HEAD-20563-1316539209-980.46099-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding:	binary
X-RT-Original-Encoding:	utf-8
Content-Length:	127

Download (untitled) / with headers
text/plain 127b

Making literal tags configurable would also be useful for those doing javascript templates with <script type="text/html"> tags.

# Wed Oct 17 18:22:02 2012 andrew [...] pimlott.net - Correspondence added

MIME-Version:	1.0
In-Reply-To:	<rt-3.6.HEAD-13950-1245489460-1333.46099-0-0 [...] rt.cpan.org>
X-Mailer:	MIME-tools 5.427 (Entity 5.427)
Content-Disposition:	inline
References:	<rt-3.6.HEAD-13950-1245489460-1333.46099-0-0 [...] rt.cpan.org>
Content-Type:	text/plain; charset="UTF-8"
Message-ID:	<rt-3.8.HEAD-31049-1350512522-1469.46099-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding:	binary
From:	andrew [...] pimlott.net
X-RT-Original-Encoding:	utf-8
Content-Length:	466

Download (untitled) / with headers
text/plain 466b

On Sat Jun 20 05:17:40 2009, GAAS wrote: Show quoted text

> > I'd like to at least be able to turn on parsing for iframes, even if

> it

> > is off by default.

> > I see the point if you need to emulate the behaviour of very old > browsers.

What is the point of not parsing the content of iframes? I can't find any justification, and it seems at odds both with the spec and user expectations. Removing this special case would make HTML::Parser simpler and more uniform. Andrew

# Thu Oct 18 18:09:53 2012 GAAS [...] cpan.org - Correspondence added

MIME-Version:	1.0
In-Reply-To:	<rt-3.8.HEAD-31049-1350512522-1469.46099-0-0 [...] rt.cpan.org>
X-Mailer:	MIME-tools 5.427 (Entity 5.427)
Content-Disposition:	inline
References:	<rt-3.6.HEAD-13950-1245489460-1333.46099-0-0 [...] rt.cpan.org> <rt-3.8.HEAD-31049-1350512522-1469.46099-0-0 [...] rt.cpan.org>
Content-Type:	text/html; charset="UTF-8"
Message-ID:	<rt-3.8.HEAD-30680-1350598193-1049.46099-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding:	binary
X-RT-Original-Encoding:	utf-8
Content-Length:	106

Download (untitled) / with headers
text/html 106b

I explained the point just above the text you quoted. What's "the spec" you'r refering to?

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

X-RT-Interface:	REST
MIME-Version:	1.0
X-Mailer:	MIME-tools 5.504 (Entity 5.504)
RT-Message-ID:	<rt-4.0.18-27437-1598294200-6.46099-0-0 [...] rt.cpan.org>
Content-Type:	text/plain; charset="utf-8"
Content-Disposition:	inline
Content-Transfer-Encoding:	binary
Content-Length:	81

Download (untitled) / with headers
text/plain 81b

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

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