don't reject on error

because we ignore it anyways, and it makes an unrelated unit test fail
krombel · Mar 12, 2019 · 8c2001a · 8c2001a
1 parent 79ca235
commit 8c2001a
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/crypto/olmlib.js b/src/crypto/olmlib.js
@@ -202,7 +202,7 @@ module.exports.ensureOlmSessionsForDevices = async function(
         );
     } catch (e) {
         for (const resolver of Object.values(resolveSession)) {
-            resolver.reject(e);
+            resolver.resolve();
         }
         logger.log("failed to claim one-time keys", e, devicesWithoutSession);
         throw e;
@@ -237,7 +237,7 @@ module.exports.ensureOlmSessionsForDevices = async function(
                 const msg = "No one-time keys (alg=" + oneTimeKeyAlgorithm +
                       ") for device " + userId + ":" + deviceId;
                 logger.warn(msg);
-                resolveSession[key].reject(new Error(msg));
+                resolveSession[key].resolve();
                 continue;
             }
 
@@ -248,7 +248,7 @@ module.exports.ensureOlmSessionsForDevices = async function(
                     resolveSession[key].resolve(sid);
                     result[userId][deviceId].sessionId = sid;
                 }, (e) => {
-                    resolveSession[key].reject(e);
+                    resolveSession[key].resolve();
                     throw e;
                 }),
             );