Length of Group Messages can Sometimes Exceed Ideal Character Length #345

Closed
opened 2022-01-27 17:32:21 +00:00 by sarah · 2 comments
Owner

macOS 12.2 (21D48)
Cwtch 1.5.2-71-g2bff

Steps:

  1. Create a new group chat.
  2. Post a message of 1800 characters.

What happened:
Text is not posted and instead multiple instances of the message bellow are logged.

flutter: Tail recursive call to messageHandler called. This should be a rare event. If you see multiples of this log over a short period of time please log it as a bug.

Expected result:
Text is posted. 

macOS 12.2 (21D48) Cwtch 1.5.2-71-g2bff Steps: 1. Create a new group chat. 2. Post a message of 1800 characters. What happened: Text is not posted and instead multiple instances of the message bellow are logged. flutter: Tail recursive call to messageHandler called. This should be a rare event. If you see multiples of this log over a short period of time please log it as a bug. Expected result: Text is posted. 
Author
Owner

Replicated this on Linux (but did not result in message handler calls) - This happens because the encryption of the message is dependent on the encryption processs / padding and the eventualy base64 encoding of the message uploaded to the server.

Length checks can fail at any point between these steps and so the UI should be conservative on length for now. Hybrid Groups should remove distinction between p2p and group message length

Replicated this on Linux (but did not result in message handler calls) - This happens because the encryption of the message is dependent on the encryption processs / padding and the eventualy base64 encoding of the message uploaded to the server. Length checks can fail at any point between these steps and so the UI should be conservative on length for now. Hybrid Groups should remove distinction between p2p and group message length
sarah added the
bug
label 2022-01-27 17:37:29 +00:00
Author
Owner

Fixed in #346

Fixed in #346
sarah added this to the Cwtch Beta 1.6 project 2022-01-27 21:11:17 +00:00
sarah added the
cwtch-beta-1.6
label 2022-02-09 20:29:03 +00:00
sarah closed this issue 2022-02-09 20:29:06 +00:00
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cwtch.im/cwtch-ui#345
No description provided.