Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 66288 - privoxy fails to filter marquee tags correctly
Summary: privoxy fails to filter marquee tags correctly
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo Web Application Packages Maintainers
URL: http://www.skagenfondene.no/frameset.asp
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-10-04 01:50 UTC by Ketil Malde
Modified: 2004-11-18 02:24 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ketil Malde 2004-10-04 01:50:53 UTC
On the given URL (http://www.skagenfondene.no/frameset.asp), privoxy is supposed to filter out <blink> and <marquee> tags.  From the /etc/privoxy/default.filter:

# The <BLINK> and <MARQUEE> tags were crimes! 
# 
s*</?(blink|marquee)>**ig 

Notice how this doesn't take into account marquee's with additional parameters (i.e. the regex matches exactly <marquee> but not <marquee this and that>.

Privoxy apparently fixes broken HTML, in this case by adding a </marquee> at the end of the page (since it succeeded in removing the closing tag).  The result is that the whole page is scrolling, not just the top line.


Reproducible: Always
Steps to Reproduce:
Load the page with privoxy enabled.
Actual Results:  
See the 'details'.

Expected Results:  
See the 'details'.

I fixed it by adding a line to /etc/privoxy/default.filter:

   # The <BLINK> and <MARQUEE> tags were crimes! 
   # 
   s*</?(blink|marquee)>**ig 
 + s*<marquee[^<]+>**ig 

There may exist more elegant ways, of course.
Comment 1 Torsten Veller (RETIRED) gentoo-dev 2004-11-17 09:11:37 UTC
Which version of privoxy are you using?

privoxy-3.0.3 contains a corrected version:
| # The <BLINK> and <MARQUEE> tags were crimes!
| #
| s-</?(blink|marquee).*>--sigU

Can this one be closed?
Comment 2 Ketil Malde 2004-11-18 02:24:54 UTC
I haven't tested it, but it looks okay to me.  I'll close it, then.