[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> <div align="right"><font
+ size="-2">[ <cleaned_tag target="_blank" style="text-decoration: none;" href="http://bbc.internetia.pl/news.html?info=21887">więcej
+ online...</a> ]</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.
+
+