From ec26b16ddf0626bc41d33a94b4e37ed2b22b9014 Mon Sep 17 00:00:00 2001 From: Travis Ralston Date: Wed, 28 Apr 2021 09:24:19 -0600 Subject: [PATCH] Raise logging dramatically to chase pending event errors For https://github.com/vector-im/element-web/issues/17090 and similar issues This logging is expected to exist no longer than a day. --- src/client.js | 3 ++- src/logger.ts | 3 ++- src/models/room.js | 3 +++ 3 files changed, 7 insertions(+), 2 deletions(-) diff --git a/src/client.js b/src/client.js index 63514c2e5..f44a4c29d 100644 --- a/src/client.js +++ b/src/client.js @@ -2858,7 +2858,8 @@ MatrixClient.prototype._sendCompleteEvent = function(roomId, eventObject, txnId, } const type = localEvent.getType(); - logger.log(`sendEvent of type ${type} in ${roomId} with txnId ${txnId}`); + // XXX: @@TR: Return to .log (for https://github.com/vector-im/element-web/issues/17090) + logger.trace(`sendEvent of type ${type} in ${roomId} with txnId ${txnId}`); localEvent.setTxnId(txnId); localEvent.setStatus(EventStatus.SENDING); diff --git a/src/logger.ts b/src/logger.ts index 005d27665..7286c8be9 100644 --- a/src/logger.ts +++ b/src/logger.ts @@ -60,7 +60,8 @@ log.methodFactory = function(methodName, logLevel, loggerName) { * Can be tailored down to specific use cases if needed. */ export const logger: PrefixedLogger = log.getLogger(DEFAULT_NAMESPACE); -logger.setLevel(log.levels.DEBUG); +// XXX: @@TR: Return to DEBUG (for https://github.com/vector-im/element-web/issues/17090) +logger.setLevel(log.levels.TRACE); interface PrefixedLogger extends Logger { withPrefix?: (prefix: string) => PrefixedLogger; diff --git a/src/models/room.js b/src/models/room.js index 7bb263320..f3551b8f0 100644 --- a/src/models/room.js +++ b/src/models/room.js @@ -1257,6 +1257,9 @@ Room.prototype.addPendingEvent = function(event, txnId) { txnId); } + // XXX: @@TR: Remove logging (for https://github.com/vector-im/element-web/issues/17090) + logger.trace(`addPendingEvent called for txn ${txnId} and event type ${event.getType()}`); + // call setEventMetadata to set up event.sender etc // as event is shared over all timelineSets, we set up its metadata based // on the unfiltered timelineSet.