From deb2f38ab33b74567bba6be879440da1d26f5a4b Mon Sep 17 00:00:00 2001
From: Christian Paul <christianp@matrix.org>
Date: Mon, 10 Aug 2020 20:54:53 +0200
Subject: [PATCH] Test for type and state_key being strings, not being truthy

---
 src/components/state-lookup.ts | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/src/components/state-lookup.ts b/src/components/state-lookup.ts
index 99a65817..19f41248 100644
--- a/src/components/state-lookup.ts
+++ b/src/components/state-lookup.ts
@@ -110,9 +110,7 @@ export class StateLookup {
             return es[eventType][stateKey] || null;
         }
 
-        return Object.keys(es[eventType]).map(function(skey) {
-            return es[eventType][skey];
-        });
+        return Object.keys(es[eventType]).map(skey => es[eventType][skey]);
     }
 
     private async getInitialState(roomId: string): Promise<StateLookupRoom> {
@@ -195,8 +193,8 @@ export class StateLookup {
             // Reject - unexpected content type
             return;
         }
-        if (!event.type || !event.state_key) {
-            // Reject - missing keys
+        if (typeof event.type !== "string" || typeof event.state_key !== "string") {
+            // Reject - invalid keys
             return;
         }
         // blunt update