Quite a few uses of strange if/else if/else's

[finnhodgkin]

In a few places in your codebase you're nesting if statements within the else block of an if statement, why not just use else if?

if (error) {
        console.log('error checking username', err);
      } else {
        if (result == true) {
          res.writeHead(200, { 'Content-Type': 'text/html' });
          res.end(JSON.stringify('username already exists'));
        } else {

vs

if (error) {
  // HANDLE THE ERRORS
} else if (result == true) {
  res.writeHead(200, { 'Content-Type': 'text/html' });
  res.end(JSON.stringify('username already exists'));
} else {
 THE REST
}

or even

if (error) {
  return // HANDLE THE ERRORS
} else if (result == true) {
  res.writeHead(200, { 'Content-Type': 'text/html' });
  return res.end(JSON.stringify('username already exists'));
}
// THE REST OF YOUR CODE NOT IN A BLOCK.