Fix the decoding of ?next=

fec28336 · Ned Batchelder · Anurag Ramdasan · dea5c7d3 · fec28336
Commit fec28336 authored Jul 23, 2014 by Ned Batchelder Committed by Anurag Ramdasan Jul 24, 2014
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

lms/templates/login.html
+2 -2

No files found.
--- a/lms/templates/login.html
+++ b/lms/templates/login.html
@@ -54,8 +54,8 @@
          var next = u.split("next=")[1];
          if (next != undefined) {
            // if next is undefined, decodeURI returns "undefined" causing a bad redirect.
-            next = decodeURI(next); 
-	  }		
+            next = decodeURIComponent(next);
+          }
          if (next && !isExternal(next)) {
            location.href=next;
          }  else if(json.redirect_url){