spreed
spreed copied to clipboard
"to" is duplicated when sending messages in "sendCallMessage"
How to use GitHub
- Please use the 👍 reaction to show that you are affected by the same issue.
- Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
- Subscribe to receive notifications on status change and new comments.
Steps to reproduce
- Setup external signaling.
- Start typing in chat.
- Look at messages being sent for the typing indication.
Expected behaviour
The session id the recipient should only be included once.
Actual behaviour
The session id of the recipient is used in the recipient and also included in the data:
https://github.com/nextcloud/spreed/blob/ff123623e76de5ba85fcd97898c07b305d5ca3ab/src/utils/signaling.js#L932-L941
It should not be part of data as the message will only be sent to the session in recipient and the recipient knows the own id.