refactor: empty hashed keys in subscription match all entities

dojoengine · Jul 7, 2024 · a76e56f · a76e56f
1 parent 2e04f54
commit a76e56f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/crates/torii/grpc/src/server/subscriptions/entity.rs b/crates/torii/grpc/src/server/subscriptions/entity.rs
@@ -109,7 +109,7 @@ impl Service {
             // matches the key pattern of the subscriber.
             match &sub.keys {
                 Some(EntityKeysClause::HashedKeys(hashed_keys)) => {
-                    if !hashed_keys.contains(&hashed) {
+                    if !hashed_keys.is_empty() && !hashed_keys.contains(&hashed) {
                         continue;
                     }
                 }

diff --git a/crates/torii/grpc/src/server/subscriptions/event_message.rs b/crates/torii/grpc/src/server/subscriptions/event_message.rs
@@ -108,7 +108,7 @@ impl Service {
             // matches the key pattern of the subscriber.
             match &sub.keys {
                 Some(EntityKeysClause::HashedKeys(hashed_keys)) => {
-                    if !hashed_keys.contains(&hashed) {
+                    if !hashed_keys.is_empty() && !hashed_keys.contains(&hashed) {
                         continue;
                     }
                 }