summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJames Moger <james.moger@gitblit.com>2014-11-04 17:12:00 -0500
committerJames Moger <james.moger@gitblit.com>2014-11-04 17:12:00 -0500
commitb5010f9f54ae2d32cf1c86e7b7129aa2d6633053 (patch)
treef927ef9b3308d9ae944fad9bdab6d7109b9ff583
parentee116b1bb896d723894882f721e2264f9b9c0b1c (diff)
downloadgitblit-b5010f9f54ae2d32cf1c86e7b7129aa2d6633053.tar.gz
gitblit-b5010f9f54ae2d32cf1c86e7b7129aa2d6633053.zip
Whitelist the "target" link attribute in the XSS filter
-rw-r--r--releases.moxie2
-rw-r--r--src/main/java/com/gitblit/utils/JSoupXssFilter.java2
2 files changed, 3 insertions, 1 deletions
diff --git a/releases.moxie b/releases.moxie
index 2bca1a74..d98c2d41 100644
--- a/releases.moxie
+++ b/releases.moxie
@@ -18,6 +18,7 @@ r27: {
- Fix exception when viewing a ticket with a patchset where the integration branch does not exist (issue-521, ticket-212)
- Fix exception when deleting a repository using the FileTicketService (issue-522, ticket-213)
- Do not inject team repository permissions as explicit user permissoins when editing a user (issue-462, ticket-214)
+ - Whitelist the target link attribute in the XSS filter (ticket-216)
changes:
- Replaced Dagger with Guice (ticket-80)
- Use release name as root directory in Gitblit GO artifacts (ticket-109)
@@ -41,6 +42,7 @@ r27: {
- Florian Zschocke
- Paul Martin
- razzard
+ - Alexander Zabluda
}
#
diff --git a/src/main/java/com/gitblit/utils/JSoupXssFilter.java b/src/main/java/com/gitblit/utils/JSoupXssFilter.java
index b5ac59f6..aec22411 100644
--- a/src/main/java/com/gitblit/utils/JSoupXssFilter.java
+++ b/src/main/java/com/gitblit/utils/JSoupXssFilter.java
@@ -73,7 +73,7 @@ public class JSoupXssFilter implements XssFilter {
"sub", "sup", "table", "tbody", "td", "tfoot", "th", "thead", "tr", "tt", "u",
"ul", "var")
- .addAttributes("a", "class", "href", "style", "title")
+ .addAttributes("a", "class", "href", "style", "target", "title")
.addAttributes("blockquote", "cite")
.addAttributes("col", "span", "width")
.addAttributes("colgroup", "span", "width")