Clean up MR

io/userRegistration.go

@@ -41,26 +41,16 @@ func registerUser(msg *pb.UDBUserRegistration, permPublicKey *rsa.PublicKey,
 			"Please try again")
 	}
 
-	// Check if username is taken
-	err = store.CheckUser(username, uid)
-	if err != nil {
+	// Check if the username is banned
+	if bannedManager.IsBanned(username) {
+		// Return same error message as if the user was already taken
 		return &messages.Ack{}, errors.Errorf("Username %s is already taken. "+
 			"Please try again", username)
 	}
 
-	//	todo: consider flattening a username
-	// 	such that for username "admin" which we want banned,
-	// 	these usernames are also banned: ADMIN, AdMiN, Ádmíñ
-	// flattenedUsername := flatten(username)
-	// if store.IsBanned(flattenedUsername) {
-	//		// Return same error message as if the user was already taken
-	//		return &messages.Ack{}, errors.Errorf("Username %s is already taken. "+
-	//			"Please try again", username)
-	//	}
-
-	// Check if the username is banned
-	if bannedManager.IsBanned(username) {
-		// Return same error message as if the user was already taken
+	// Check if username is taken
+	err = store.CheckUser(username, uid)
+	if err != nil {
 		return &messages.Ack{}, errors.Errorf("Username %s is already taken. "+
 			"Please try again", username)
 	}