Clarify expiry of security rules for banned users (#167)

Fixes #158
ekzhang · Dec 22, 2024 · 0ac422b · 0ac422b
1 parent 7b5bdcc
commit 0ac422b
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/database.rules.json b/database.rules.json
@@ -2,7 +2,7 @@
   "rules": {
     "games": {
       "$gameId": {
-        ".read": "auth != null && !root.child('users').child(auth.uid).child('banned').exists()",
+        ".read": "auth != null && (!root.child('users').child(auth.uid).child('banned').exists() || root.child('users').child(auth.uid).child('banned').val() < now)",
         "startedAt": {
           ".write": "auth != null && auth.uid == data.parent().child('host').val() && !data.exists() && newData.exists()",
           ".validate": "newData.isNumber() && newData.val() == now"
@@ -102,7 +102,7 @@
     },
     "chats": {
       "$chatId": {
-        ".read": "auth != null && !root.child('users').child(auth.uid).child('banned').exists()",
+        ".read": "auth != null && (!root.child('users').child(auth.uid).child('banned').exists() || root.child('users').child(auth.uid).child('banned').val() < now)",
         ".indexOn": ["time", "user"],
         "$messageId": {
           ".write": "auth != null && ((!data.exists() && newData.exists()) || (data.exists() && root.child('users').child(auth.uid).child('admin').val() == true))",
@@ -123,7 +123,7 @@
       }
     },
     "lobbyChat": {
-      ".read": "auth != null && !root.child('users').child(auth.uid).child('banned').exists()",
+      ".read": "auth != null && (!root.child('users').child(auth.uid).child('banned').exists() || root.child('users').child(auth.uid).child('banned').val() < now)",
       ".indexOn": ["time", "user"],
       "$messageId": {
         ".write": "auth != null && ((!data.exists() && newData.exists()) || (data.exists() && root.child('users').child(auth.uid).child('admin').val() == true))",