无法在 Firebase Firestore 安全规则中使用逻辑 OR 运算符 - 条件始终评估为“假”

Question

如果登录用户是经理或记录不存在，我想允许写入/companies/{company}。

我有以下安全规则：

service cloud.firestore {
  match /databases/{database}/documents {
    function isManager() {
      return get(/databases/$(database)/documents/managers/$(request.auth.uid)).data.id == request.auth.uid;
    }

    function companyExists(company) {
      return exists(/databases/$(database)/documents/companies/$(company));
    }

    match /{document=**} {
      allow read, write: if false;
    }

    match /companies/{company} {
      allow read: if resource.data.isActive == true || isManager();
      allow write: if !companyExists(company) || isManager();
    }
  }
}

下面这行有问题：

allow write: if !companyExists(company) || isManager();

即使!companyExists(company) 函数的计算结果为true，此条件的计算结果始终为false。

我确信 companyExists(company) 和 isManager() 函数可以按预期工作，因为我一直在单独测试它们。

例如，如果我使用以下规则：

allow write: if !companyExists(company);

如果公司记录不存在，则表达式的计算结果为 true。与isManager() 函数相同。如果登录用户的 ID 存在于 managers 集合中，则返回 true。只有在两个函数之间使用|| 运算符时才会出现该问题。

Answer 1

没用：

allow read: if request.auth.uid != null || if request.auth.uid == "VacOqosKJhPjxQaCf93wGVjc8Fy2"

这样就可以了：

allow read: if request.auth.uid != null;
allow read: if request.auth.uid == "GePV7U9kDmfThLVXzks6nn2HgXn2";

Answer 2

您似乎希望允许所有人创建公司，但只有经理可以编辑它们。这可以通过更简单的方式实现：

match /companies/{company} {
  allow read: if resource.data.isActive == true || isManager();
  allow create;
  allow update, delete: if isManager();
}

它也会更便宜，因为您不需要执行 get 请求来检查它是否存在。

此外，还可以使用自定义声明而不是文档来重写 isManager 的逻辑，这样您还可以保存另一个读取。这超出了这个问题的范围，但如果您需要更多详细信息，请发表评论。