Skip to content
This repository has been archived by the owner on Feb 12, 2023. It is now read-only.

Message splitting fails for group messages #5760

Closed
sudden6 opened this issue Aug 2, 2019 · 0 comments · Fixed by #5761
Closed

Message splitting fails for group messages #5760

sudden6 opened this issue Aug 2, 2019 · 0 comments · Fixed by #5761
Labels
C-bug The issue contains a bug report
Milestone
v1.17.0

Comments

@sudden6
Copy link
Member

sudden6 commented Aug 2, 2019

Brief Description

OS: Windows / OS X / Linux (include version and/or distro)
Commit hash: b2e7a7f
toxcore: 0.2.10
Qt: 5.9.4

Reproducible: Always

Steps to reproduce
  1. Send a very long message (~4000 characters) without spaces to a group e.g. 1 A, 3999 B
Observed Behavior

The first message ABBBBBB.... is received correctly for all group peers the rest is marked as Message failed to send on the sender side

Expected Behavior

The message is split into smaller parts and all group peers receive the message
@sudden6 sudden6 added the C-bug The issue contains a bug report label Aug 2, 2019
@sudden6 sudden6 added this to the v1.17.0 milestone Aug 2, 2019
Kribylet added a commit to Kribylet/qTox that referenced this issue Aug 2, 2019
@Kribylet
fix(groups): reduce group message size limit by 50
e893e35 
This commit fixes errors when sending large group messages stemming from inconsistencies in reported/true max message size in c-toxcore

Fixes qTox#5760
@Kribylet Kribylet mentioned this issue Aug 2, 2019
Merged
1 task
Kribylet added a commit to Kribylet/qTox that referenced this issue Aug 2, 2019
@Kribylet
fix(groups): reduce group message size limit by 50
fa7c4a2 
This commit fixes errors when sending large group messages stemming from inconsistencies in reported/true max message size in c-toxcore

Fixes qTox#5760
Kribylet added a commit to Kribylet/qTox that referenced this issue Aug 2, 2019
@Kribylet
fix(groups): reduce group message size limit by 50
e3f6f31 
This commit fixes errors when sending large group messages stemming from inconsistencies in reported/true max message size in c-toxcore

Fixes qTox#5760
Kribylet added a commit to Kribylet/qTox that referenced this issue Aug 2, 2019
@Kribylet
fix(groups): reduce group message size limit by 50
45f39fd 
This commit fixes errors when sending large group messages stemming from inconsistencies in reported/true max message size in c-toxcore

Fixes qTox#5760
Kribylet added a commit to Kribylet/qTox that referenced this issue Aug 2, 2019
@Kribylet
fix(groups): reduce group message size limit by 50
6c77d57 
This commit fixes errors when sending large group messages stemming from inconsistencies in reported/true max message size in c-toxcore

Fixes qTox#5760
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
C-bug The issue contains a bug report
Projects
None yet
Milestone
v1.17.0
Development

Successfully merging a pull request may close this issue.

fix(groups): reduce group message size limit by 50 Kribylet/qTox
1 participant
@sudden6