[Bug 877] Changed - Cleaning style attribute for HTML documents [patch]

bugs@bugs.horde.org bugs@bugs.horde.org
Wed, 20 Feb 2002 05:35:56 -0400


http://bugs.horde.org/show_bug.cgi?id=877

*** shadow/877	Tue Feb 19 12:12:06 2002
--- shadow/877.tmp.9496	Wed Feb 20 05:35:56 2002
***************
*** 75,77 ****
--- 75,93 ----
  ------- Additional Comments From chuck@horde.org  02/19/02 12:11 -------
  Patch? I'm not going to open up more of a security hole just to allow more bits 
  of html.
+ 
+ ------- Additional Comments From dexter@top.pl  02/20/02 05:35 -------
+ Why IMP leaves <a> tag in some cases and cleans this tag in others?
+ This is inconistency. The same HTML document (newsletter):
+ 
+ <font size="-1" face="MS Sans Serif, Verdana, Helvetica, Arial, Sans-Serif">
+ Zapraszamy do subskrybcji sekcji <a target="_blank" href="http://bbc.internetia.pl/biuletyn.html"><B>flashnews</B></a>.
+ <br><font size="-1" face="MS Sans Serif, Verdana, Helvetica, Arial, Sans-Serif">Obywatele Korei Północnej zostali zaangażowani w organizację rocznicy 
+ przyjścia na świat ich przywódcy, o której już mówi się jako o "najhuczniejszych urodzinach świata".</font> &nbsp;&nbsp;<div align="right"><font 
+ size="-2">[&nbsp;<cleaned_tag target="_blank" style="text-decoration: none;" href="http://bbc.internetia.pl/news.html?info=21887">więcej 
+ online...</a>&nbsp;]</font>
+ 
+ The second <a> tag just contains style attribute which could be removed.
+ Ot the other way each <a> tag should be removed, not just some of them.
+ 
+