Removes username check which doesn't do any good #402

Merged
merged 1 commit into from Aug 20, 2012

Conversation

Projects
None yet
2 participants
Contributor

Tenzer commented Aug 18, 2012

The username is already checked at https://github.com/nodester/nodester/blob/75cbc48b7d/lib/user.js#L136-140 during user creation, and since change of username isn't possible, it doesn't make sense to check the username anywhere else since it's part of the authentication.

alejandro added a commit that referenced this pull request Aug 20, 2012

Merge pull request #402 from Tenzer/unnecessary-username-check
Removes username check which doesn't do any good

@alejandro alejandro merged commit d9e2d28 into nodester:master Aug 20, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment