Firebase Cloud Function 未更新用户的电话号码

Question

我认为这不是 Firebase 特有的。它可能与等待/承诺有更多关系，但我无法弄清楚。

编写 Firebase 函数以允许授权用户更新另一个用户帐户。看起来很简单。请注意 cmets。请注意，1）旧电话号码未定义（如预期），2）新/传入的电话号码是合法的，3）分配后，“新”密码仍然未定义。为什么？！？！？

我错过了什么？？？ （对不起，我不能让这个问题更简洁！）

module.exports = functions.https.onCall(async (data, context) => {
  var targetUser = await admin.auth().getUser(data.targetUser.uid)
  // Following rule might not make sense! But it works. We know cuz the following later logs output correctly!
  if (
    Boolean(targetUser) && Boolean(context.auth) &&
    Boolean(context.auth.token.companyToken) &&
    (Boolean(context.auth.token.isAdmin) || context.auth.email === targetUser.email)
  ) {
    // Log output appears from the following!
    console.log(`Phone was ${targetUser.phoneNumber}`) // "Phone was undefined" - expected!
    console.log(`Passed-in Phone is ${data.targetUser.phoneNumber}`) // "Phone is (402)555-1212" - Yep!
    // Again, ignore the logic here. It's working! I get the embedded log output
    if (context.auth.email === targetUser.email || context.auth.token.companyToken === targetUser.customClaims.companyToken) {
      // the following appears to not execute! I expect targetUser.phoneNumber to be updated!
      // ... it SHOULD become data.targetUser.phoneNumber (i.e. "(402)555-1212")
      targetUser.phoneNumber = (data.targetUser.phoneNumber ? data.targetUser.phoneNumber : targetUser.phoneNumber)
      console.log(`Phone NOW ${targetUser.phoneNumber}`) // "Phone is NOW undefined"
      // WTF?!?!? It's still undefined?!?!?
      // THIS IS MY PROBLEM RIGHT HERE! WHY IS phoneNumber STILL UNDEFINED?!?!?!
      // You can ignore the rest...
      return updateUser(targetUser).then(() => {
        return {
          success: true
        }
      })
      .catch((e) => {
        return {
        // yadda yadda
        }
      })
    }
  }
  throw new functions.https.HttpsError(
    "not-authorized",
    "Requestor is not authorized to perform this operation on the target user."
  )
})

Answer 1

不是您的代码的问题。变量声明的问题，请用 let 代替 var，你已经在路上了。

module.exports = functions.https.onCall(async (data, context) => {
  //var targetUser = await admin.auth().getUser(data.targetUser.uid);
  // declare the targetUser with let instead of var.
  let targetUser = await admin.auth().getUser(data.targetUser.uid);
  //following later logs output correctly!
  if (
    Boolean(targetUser) && Boolean(context.auth) &&
    Boolean(context.auth.token.companyToken) &&
    (Boolean(context.auth.token.isAdmin) || context.auth.email === targetUser.email)
  ) {
    // Log output appears from the following!
    console.log(`Phone was ${targetUser.phoneNumber}`) // "Phone was undefined" - expected!
    console.log(`Passed-in Phone is ${data.targetUser.phoneNumber}`) // "Phone is (402)555-1212" - Yep!
    // Again, ignore the logic here. It's working! I get the embedded log output
    if (context.auth.email === targetUser.email || context.auth.token.companyToken === targetUser.customClaims.companyToken) {
      // the following appears to not execute! I expect targetUser.phoneNumber to be updated!
      // ... it SHOULD become data.targetUser.phoneNumber (i.e. "(402)555-1212")
      targetUser.phoneNumber = (data.targetUser.phoneNumber ? data.targetUser.phoneNumber : targetUser.phoneNumber)
      console.log(`Phone NOW ${targetUser.phoneNumber}`) // "Phone is NOW undefined"
      // WTF?!?!? It's still undefined?!?!?
      // THIS IS MY PROBLEM RIGHT HERE! WHY IS phoneNumber STILL UNDEFINED?!?!?!
      // You can ignore the rest...
      return updateUser(targetUser).then(() => {
        return {
          success: true
        }
      })
      .catch((e) => {
        return {
        // yadda yadda
        }
      })
    }
  }
  throw new functions.https.HttpsError(
    "not-authorized",
    "Requestor is not authorized to perform this operation on the target user."
  )
})

如果您仍然遇到任何问题，请告诉我我们会找到另一种方法。

Answer 2

这是因为targetUser 是UserRecord（getUser() 方法返回的 Promise 使用 UserRecord 解析）并且您不能像简单的 JavaScript 对象那样更新 UserRecord。

你需要调用updateUser()方法如下：

const newPhoneNumber = '.....';
const newUserRecord = await admin.auth().updateUser(data.targetUser.uid, {phoneNumber: newPhoneNumber})
console.log(newUserRecord.phoneNumber);

---

另外请注意，您将async/await 与then() 方法混合使用，不推荐这样做。