Support (%40|@) username (en|de)coding for repository urls and authentication

James Moger

2013-06-19 15cb305fbd441f0d7b3e7261d956472db3718417

Support (%40|@) username (en|de)coding for repository urls and authentication

 src/main/java/com/gitblit/GitBlit.java

@@ -509,7 +509,7 @@
        if (user == null) {
            user = UserModel.ANONYMOUS;
        }
        String username = UserModel.ANONYMOUS.equals(user) ? "" : user.username;
        String username = encodeUsername(UserModel.ANONYMOUS.equals(user) ? "" : user.username);

        List<RepositoryUrl> list = new ArrayList<RepositoryUrl>();
        // http/https url
@@ -735,6 +735,7 @@
            // can not authenticate empty username
            return null;
        }
        String usernameDecoded = decodeUsername(username);
        String pw = new String(password);
        if (StringUtils.isEmpty(pw)) {
            // can not authenticate empty password
@@ -743,7 +744,7 @@

        // check to see if this is the federation user
        if (canFederate()) {
            if (username.equalsIgnoreCase(Constants.FEDERATION_USER)) {
            if (usernameDecoded.equalsIgnoreCase(Constants.FEDERATION_USER)) {
                List<String> tokens = getFederationTokens();
                if (tokens.contains(pw)) {
                    // the federation user is an administrator
@@ -758,7 +759,7 @@
        if (userService == null) {
            return null;
        }
        return userService.authenticate(username, password);
        return userService.authenticate(usernameDecoded, password);
    }

    /**
@@ -966,6 +967,14 @@
        userService.logout(user);
    }

    protected String encodeUsername(String name) {
        return name.replace("@", "%40");	
    }

    protected String decodeUsername(String name) {
        return name.replace("%40", "@");	
    }
	
    /**
     * Returns the list of all users available to the login service.
     * 
@@ -999,7 +1008,8 @@
        if (StringUtils.isEmpty(username)) {
            return false;
        }
        return userService.deleteUser(username);
        String usernameDecoded = decodeUsername(username);
        return userService.deleteUser(usernameDecoded);
    }

    /**
@@ -1013,7 +1023,8 @@
        if (StringUtils.isEmpty(username)) {
            return null;
        }
        UserModel user = userService.getUserModel(username);		
        String usernameDecoded = decodeUsername(username);
        UserModel user = userService.getUserModel(usernameDecoded);		
        return user;
    }