如何防止Firebase中出现重复的用户属性?

时间:2014-08-13 19:24:14

标签: angularjs firebase firebase-realtime-database angularfire firebase-security

我正在使用FirebaseSimpleLogin来创建用户并处理身份验证。

当我尝试通过$createUser()方法创建一个简单登录的新用户时,如果已经使用了电子邮件地址,firebase将不会创建用户。但是,在我创建用户并使用$set()作为密钥后,我还使用user.uid将我创建的用户保存到我的firebase。当尝试写入数据库时​​,即使用户名不是唯一的,firebase也会保存记录,因为简单登录只需要电子邮件和密码。那么,当用户名不被用作用户对象的密钥时,如何验证用户名是否唯一?

我正在创建这样的新用户:

$scope.createUser = function() {
  $scope.auth.$createUser('trinker@gmail.com', 'password').then(function(user, err) {
    if (!err) {
      ref.child('users/' + user.uid).set({
        email: user.email,
        username: user.username
      });
      console.log("success!");
    }else{
      console.log(err.message);
    }
  });
}

我的用户对象如下所示:

{
  "users" : {
    "simplelogin:28" : {
      "email" : "trinker@gmail.com",
      "username" : "jtrinker"
    },
    "simplelogin:30" : {
      "email" : "test@gmail.com",
      "username" : "jtrinker"
    }
  }
}

}

我需要使用uid作为每个用户的密钥,但我仍然需要用户名是唯一的。

如果一个对象中的属性对于另一个对象内的属性不是唯一的,我怎么能阻止firebase保存记录?

2 个答案:

答案 0 :(得分:30)

首先,如果用户已经拥有username,这是唯一的,而且这不会消失,我建议您放弃使用简单的登录uid。正如你在这里发现的那样,这只会产生试图在两者之间来回翻转的问题。使用creating your own tokens等工具调查firebase-passport-login,然后按username存储记录。

但由于这不是你的问题,让我们解决这个问题,因为你们可能想要继续前进,进入我已经多次通过的双重身份的棘手荆棘。

要使用户名唯一,请存储用户名索引。

/users/$userid/username/$username
/usernames/$username/$userid

要确保它们是唯一的,请在usernames / path中的用户ID上添加如下安全规则,这样可确保每个用户只能分配一个用户,并且该值是用户的ID:

".write": "newData.val() === auth.uid && !data.exists()"

现在通过将以下内容添加到users / record中的用户名来强制它们匹配:

"users": {
   "$userid": {
      "username": {
         ".validate": "root.child('usernames/'+newData.val()).val() === $userid"
      }
   }
}

这将确保ID是唯一的。请注意阅读权限。您可能希望完全避免这些,因为您不希望任何人查找私人电子邮件或用户名。 Something like I demonstrated in support用于保存这些是理想的。

这里的想法是你尝试分配用户名和电子邮件,如果它们失败,那么它们已经存在并且属于另一个用户。否则,您将它们插入到用户记录中,现在让用户通过uid和电子邮件编入索引。

为了符合SO协议,这里是该要点的代码,可以通过链接更好地阅读:

var fb = new Firebase(URL);

function escapeEmail(email) {
   return email.replace('.', ',');
}

function claimEmail(userId, email, next) {
   fb.child('email_lookup').child(escapeEmail(email)).set(userId, function(err) {
      if( err ) { throw new Error('email already taken'); }
      next();
   });
}

function claimUsername(userId, username, next) {
   fb.child('username_lookup').child(username).set(userId, function(err) {
      if( err ) { throw new Error('username already taken'); }
      next();
   });   
}

function createUser(userId, data) {
   claimEmail(userId, data.email, claimUsername.bind(null, userId, data.username, function() {
      fb.child('users').child(userId).set(data);
   );   
}

规则:

{
  "rules": {
     "users": {
        "$user": {
           "username": {
               ".validate": "root.child('username_lookup/'+newData.val()).val() === auth.uid"
           },
           "email": {
               ".validate": "root.child('email_lookup').child(newData.val().replace('.', ',')).val() === auth.uid"
           }
        }
     },

     "email_lookup": {
        "$email": {
           // not readable, cannot get a list of emails!
           // can only write if this email is not already in the db
           ".write": "!data.exists()",
           // can only write my own uid into this index
           ".validate": "newData.val() === auth.uid"
        }
     },
     "username_lookup": {
        "$username": {
           // not readable, cannot get a list of usernames!
           // can only write if this username is not already in the db
           ".write": "!data.exists()",

           // can only write my own uid into this index
           ".validate": "newData.val() === auth.uid"
        }
     },
  }
}

答案 1 :(得分:3)

Wouldn't it be easier to just use the security rules to check for it's existence? I have mine set up as follows:

"usernames": {
  "$usernameid": {
    ".read": "auth != null",
        ".write": "auth != null  && (!data.exists() || !newData.exists())"
    }
    }

This allows the write if the username doesn't exist. I believe I got this directly from the Firebase docs.