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

Question

目前，我在 functions.auth.user().onCreate 触发器上有一个 Firebase 云函数 initUser。它从身份验证提供程序(仅限 GitHub)获取用户的显示名称，并在实时数据库中为用户创建一个条目。我会说这会像描述的那样工作大约三分之一。对于另外两次从 event 获取 user 和 displayName 返回 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 获取用户显示名称。

注意:我在其他地方看到有人建议添加延迟，不接受这个作为答案。

编辑:建议的答案是对代码的改进，但它不是根本原因，因为错误仍然存​​在。为了澄清这个问题，我如何始终避免 github 中的用户显示名称 undefined 特别是在 initUser 云函数中。

Answer 1

将数据写入 Firebase(和大多数现代网络 API)是异步发生的。当您调用 admin.database()....set(...) 时，它只会开始写入数据，可能需要一些时间才能完成写入操作.但是，只要您在代码末尾返回 true，Cloud Functions 系统就会假定您的代码已完成，并且可以自由取消任何剩余的操作。这种竞争条件是写入丢失的最常见原因。

要修复竞争条件，您需要返回一个 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 .