Merge pull request #773 from Martii/adminUTFIssues

Fix flagging and AuthAs Auto-merge
OpenUserJS · Oct 20, 2015 · 1836ce3 · 1836ce3
2 parents b9f1765 + 666c867
commit 1836ce3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/controllers/admin.js b/controllers/admin.js
@@ -474,6 +474,6 @@ exports.authAsUser = function (aReq, aRes, aNext) {
 
     aReq.session.user = user;
 
-    aRes.redirect(user.userPageUrl);
+    aRes.redirect(encodeURI(user.userPageUrl));
   });
 };
diff --git a/controllers/user.js b/controllers/user.js
@@ -1481,7 +1481,7 @@ exports.flag = function (aReq, aRes, aNext) {
     if (aErr || !aUser) { return aNext(); }
 
     fn(User, aUser, aReq.session.user, function (aFlagged) { // NOTE: Inline function here
-      aRes.redirect('/users/' + username);
+      aRes.redirect('/users/' + encodeURI(username));
     });
   });
 };