Default escapeXml behavior should be configurable for environments that want to
escape by default
-------------------------------------------------------------------------------------------------
Key: DISPL-664
URL: http://jira.codehaus.org/browse/DISPL-664
Project: DisplayTag
Issue Type: Improvement
Components: Configuration, Tag Library
Affects Versions: 1.2
Reporter: Dan L
DISPL-80 (http://jira.codehaus.org/browse/DISPL-80) added the escapeXml
attribute to the display:column tag. However, it was added so that columns
don't escape by default -- you need to add the property to enable the escaping.
There are many security and application development arguments that could be
made in favor of an escape-by-default strategy -- e.g. every column should
escape xml by default unless explicitly told otherwise because if the developer
is forced to explicitly turn off escaping, he is forced to consider whether he
has addressed all potential html injection attack vectors.
However, I don't want to turn this into a
philosophical/backwards-compatibility/whatever issue. We can keep the
don't-escape-by-default behavior, but at the very least, this behavior should
be customizable to support development environments that take a more
conservative view to this type of thing. An overridable property in
displaytag.properties to control the default escaping behavior would be the way
to go.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
------------------------------------------------------------------------------
Increase Visibility of Your 3D Game App & Earn a Chance To Win $500!
Tap into the largest installed PC base & get more eyes on your game by
optimizing for Intel(R) Graphics Technology. Get started today with the
Intel(R) Software Partner Program. Five $500 cash prizes are up for grabs.
http://p.sf.net/sfu/intelisp-dev2dev
_______________________________________________
displaytag-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/displaytag-devel
