目前,我有initUser
触发器上的Firebase云功能functions.auth.user().onCreate
。它从auth提供程序(仅GitHub)获取用户的显示名称,并在实时数据库中为用户创建一个条目。我会说这个工作描述的有三次。另外两次从user
获取displayName
然后event
返回undefined
。目前,我只是检查用户的displayName
是否为undefined
,是否将用户名设置为uid
,如果不是,我会将其设置为displayName
我是通过。
当前功能:
exports.initUser = functions.auth.user().onCreate(event => {
//Adds the Player To Users and Scores Node
const user = event.data;
const uid = user.uid;
const score = 0;
admin.database().ref('users/' + uid).child('score').set(score);
var name = "";
if(user.displayName === undefined) {
name = uid;
} else {
name = validator.escape(user.displayName);
}
admin.database().ref('users/' + uid).child('name').set(name);
admin.database().ref('scores').child(uid).set(score);
console.log("Created User: " + name);
return true;
});
所以我的问题:这是最好的方法,还是有更好的方法或已知的解决方法来始终从GitHub获取用户显示名称。
注意:我看到有人建议在其他地方添加延迟,不要接受这个作为答案。
编辑:建议的答案是对代码的改进,但由于错误仍然存在,因此不是根本原因。为了澄清问题,如何在undefined
云函数中始终如一地避免github中用户的显示名称为initUser
。
答案 0 :(得分:1)
将数据写入Firebase(以及大多数现代Web API)是异步发生的。当您致电admin.database()....set(...)
时,只有开始写入数据,写入操作完成之前可能需要一些时间。但是,只要您在代码末尾return true
,云功能系统就会假定您的代码已完成,并且可以自由取消任何剩余的操作。这种竞争条件是丢失写入的最常见原因。
要修复竞争条件,您需要返回Promise
,以便在您的代码实际完成时发出信号。在这种情况下,您有三个写操作,因此您返回一个在完成所有三个操作时解析的promise:
exports.initUser = functions.auth.user().onCreate(event => {
//Adds the Player To Users and Scores Node
var promises = [];
const user = event.data;
const uid = user.uid;
const score = 0;
promises.push(admin.database().ref('users/' + uid).child('score').set(score));
var name = "";
if(user.displayName === undefined) {
name = uid;
} else {
name = validator.escape(user.displayName);
}
promises.push(admin.database().ref('users/' + uid).child('name').set(name));
promises.push(admin.database().ref('scores').child(uid).set(score));
console.log("Created User: " + name);
return Promise.all(promises);
});
我建议您查看一些其他questions dealing with Promise.all(...)
and Cloud Functions。
答案 1 :(得分:0)
.onAuthStateChanged
可以驱动应用的用户/身份验证状态。
firebase.auth().onAuthStateChanged(function(user) { // THIS IS ASYNCH
if (user) { //USER IS SIGNED IN NOW
// NOW GET .displayName
}
else { //USER IS NOT SIGNED IN
}
});