Merge pull request #2882 from matrix-org/travis/permalink-explosion

Prevent the permalink creator from causing cascading failure
This commit is contained in:
Travis Ralston 2019-04-08 06:56:13 -06:00 committed by GitHub
commit a53d9f59fe
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -83,6 +83,14 @@ export class RoomPermalinkCreator {
}
load() {
if (!this._room || !this._room.currentState) {
// Under rare and unknown circumstances it is possible to have a room with no
// currentState, at least potentially at the early stages of joining a room.
// To avoid breaking everything, we'll just warn rather than throw as well as
// not bother updating the various aspects of the share link.
console.warn("Tried to load a permalink creator with no room state");
return;
}
this._updateAllowedServers();
this._updateHighestPlUser();
this._updatePopulationMap();