Anonymous avatar Anonymous committed 4f73e9e

add client side validation for the new username rules

Comments (0)

Files changed (1)

frontend/js/registration.js

 
 var register = {
     checkUsername:function() {
-        $('register_username_error').innerHTML = ($F('register_username').length < 4) ? "Usernames must be at least 4 characters long" : "";
+        var username_error = [];
+        if ($F('register_username').length < 4) {
+            username_error.push("Usernames must be at least 4 characters long");   
+        }
+        if (/[<>| '"]/.test($F('register_username'))) {
+            username_error.push("Usernames must not contain any of: <>| '\"");   
+        }
+        $('register_username_error').innerHTML = username_error.join(", ");
     },
     checkPassword:function() {
         $('register_password_error').innerHTML = ($F('register_password').length < 6) ? "Passwords must be at least 6 characters long" : "";
Tip: Filter by directory path e.g. /media app.js to search for public/media/app.js.
Tip: Use camelCasing e.g. ProjME to search for ProjectModifiedEvent.java.
Tip: Filter by extension type e.g. /repo .js to search for all .js files in the /repo directory.
Tip: Separate your search with spaces e.g. /ssh pom.xml to search for src/ssh/pom.xml.
Tip: Use ↑ and ↓ arrow keys to navigate and return to view the file.
Tip: You can also navigate files with Ctrl+j (next) and Ctrl+k (previous) and view the file with Ctrl+o.
Tip: You can also navigate files with Alt+j (next) and Alt+k (previous) and view the file with Alt+o.