Skip to content

Commit 6830921

Browse files
committed
Remove OTK claim timeout logging
1 parent 1bb8c2d commit 6830921

File tree

1 file changed

+0
-14
lines changed

1 file changed

+0
-14
lines changed

src/crypto/olmlib.js

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -293,18 +293,6 @@ export async function ensureOlmSessionsForDevices(
293293
const oneTimeKeyAlgorithm = "signed_curve25519";
294294
let res;
295295
let taskDetail = `one-time keys for ${devicesWithoutSession.length} devices`;
296-
// If your homeserver takes a nap here and never replies, this process
297-
// would hang indefinitely. While that's easily fixed by setting a
298-
// timeout on this request, let's first log whether that's the root
299-
// cause we're seeing in practice.
300-
// See also https://github.com/vector-im/element-web/issues/16194
301-
let otkTimeoutLogger;
302-
// XXX: Perhaps there should be a default timeout?
303-
if (otkTimeout) {
304-
otkTimeoutLogger = setTimeout(() => {
305-
log.error(`Homeserver never replied while claiming ${taskDetail}`);
306-
}, otkTimeout);
307-
}
308296
try {
309297
log.debug(`Claiming ${taskDetail}`);
310298
res = await baseApis.claimOneTimeKeys(
@@ -317,8 +305,6 @@ export async function ensureOlmSessionsForDevices(
317305
}
318306
log.log(`Failed to claim ${taskDetail}`, e, devicesWithoutSession);
319307
throw e;
320-
} finally {
321-
clearTimeout(otkTimeoutLogger);
322308
}
323309

324310
if (failedServers && "failures" in res) {

0 commit comments

Comments
 (0)