Check state key while parsing `m.call.member` state events. - Githubissues

matrix-org / matrix-js-sdk

Matrix Client-Server SDK for JavaScript

Apache License 2.0

1.55k stars 583 forks source link

Check state key while parsing `m.call.member` state events. #4265

Open toger5 opened 3 months ago

toger5 commented 3 months ago

Currently we do not check m.call.member state event keys allowing users to have multiple events for one device.

This is not to spec. We should check the key when parsing the state event here: https://github.com/matrix-org/matrix-js-sdk/blob/8ebf04ae94e3f97c8326b7be5bffc56e47655c0a/src/matrixrtc/CallMembership.ts#L45-L63

Comment with more details: https://github.com/matrix-org/matrix-js-sdk/pull/4262#discussion_r1649240390