MatrixRTCSessionManager keeps logging on error level

[finsterwalder]

@dbkr added new code to MatrixRTCSessionManager in this commit: 6836720

matrix-js-sdk/src/matrixrtc/MatrixRTCSessionManager.ts

Lines 102 to 108 in 95baccf

	private onRoomState = (event: MatrixEvent, _state: RoomState): void => {
	const room = this.client.getRoom(event.getRoomId());
	if (!room) {
	logger.error(`Got room state event for unknown room ${event.getRoomId()}!`);
	return;
	}

This keeps logging error messages. Obviously the event is processed, before the new room is known by the client.
This should be handled more gracefully, because it seams to be a "normal" situation.
And maybe it needs to be changed, so that the needed processing is done, after the room was created?

[jgarplind]

I tracked down error logs to the same place.

We don't use WebRTC, but because of this, we are still impacted:

        // NB. We initialise MatrixRTC whether we have call support or not: this is just
        // the underlying session management and doesn't use any actual media capabilities
        this.matrixRTC = new MatrixRTCSessionManager(this);

This can of course be filtered on our side, but I agree with OP that a more graceful handling from the SDK would be ideal.

[finsterwalder]

I would be willing to provide a patch, but need someone to discuss, what the "correct" solution would be.