You've already forked matrix-js-sdk
mirror of
https://github.com/matrix-org/matrix-js-sdk.git
synced 2025-11-28 05:03:59 +03:00
Make sure we don't end up calling /messages in a loop if things go weird
If we somehow end up in a situation where calling /messages returns a load of messages, but none of them are new, then currently we start calling /messages again and again in a tight loop. This is bad, so fix it.
This commit is contained in:
@@ -246,7 +246,11 @@ TimelineWindow.prototype.paginate = function(direction, size, makeRequest) {
|
|||||||
// end of timeline
|
// end of timeline
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
return self.paginate(direction, size);
|
// recurse to advance the index into the results, but before we do, set
|
||||||
|
// makeRequest=false. It's not an absolute given that the paginate
|
||||||
|
// request returned events which we can now use, but we certainly don't
|
||||||
|
// want to get stuck in a tight loop here if things start going wrong.
|
||||||
|
return self.paginate(direction, size, false);
|
||||||
});
|
});
|
||||||
tl.pendingPaginate = prom;
|
tl.pendingPaginate = prom;
|
||||||
return prom;
|
return prom;
|
||||||
|
|||||||
Reference in New Issue
Block a user