On 02/10/2011 10:08 PM, Alex Busenius wrote:
  Hi,
 On 02/10/2011 09:38 PM, sdumitriu (SVN) wrote:
  Author: sdumitriu
 Date: 2011-02-10 21:38:15 +0100 (Thu, 10 Feb 2011)
 New Revision: 34623
 Modified:
platform/xwiki-applications/trunk/administration/src/main/resources/XWiki/XWikiGroupSheet.xml
 Log:
 XAADMINISTRATION-218: CSRF token not included in add-group-member ajax request
 Fixed.
 Patch from Andreas Jonsson applied with a slight change (applied escapetool.url for
safety)
 Modified:
platform/xwiki-applications/trunk/administration/src/main/resources/XWiki/XWikiGroupSheet.xml
 ===================================================================
 ---
platform/xwiki-applications/trunk/administration/src/main/resources/XWiki/XWikiGroupSheet.xml
2011-02-10 20:37:52 UTC (rev 34622)
 +++
platform/xwiki-applications/trunk/administration/src/main/resources/XWiki/XWikiGroupSheet.xml
2011-02-10 20:38:15 UTC (rev 34623)
 @@ -171,7 +171,7 @@
             },
             addNewMember: function(uorg, input) {
               if (input) {
 -              var url = "${doc.getURL()}?xpage=adduorg&uorg=" +
encodeURIComponent(uorg) + "&name=" + encodeURIComponent(input.value);
 +              var url = "${doc.getURL()}?xpage=adduorg&uorg=" +
encodeURIComponent(uorg) + "&name=" + encodeURIComponent(input.value) +
"&form_token=$!{escapetool.url($services.csrf.getToken())}"; 
 actually, $services.csrf.getToken() returns the token in a URL-safe
 base64 encoding, so it should work fine without escapetool. 
I know, but an extra encoding doesn't break anything.
  Regards,
 Alex
>                 new Ajax.Request(url, {
>                   method: 'get',
>                   onSuccess: function(transport) {
> 
--
Sergiu Dumitriu
http://purl.org/net/sergiu/