如何从GitHub Auth一致地获取用户的显示名称

时间:2018-01-21 13:19:34

标签: javascript firebase firebase-realtime-database firebase-authentication google-cloud-functions

目前,我有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

2 个答案:

答案 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

    }
});