yplo6403 / android-rcs-ims-stack

Automatically exported from code.google.com/p/android-rcs-ims-stack
0 stars 0 forks source link

Not wanted REFER on participant adding #175

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. A (+33678699564) has started a GroupChat with B (+33787218781) and C 
(+33678567585)
2. B leaves the GroupChat
3. A adds B in the GroupChat
4. B accepts

What is the expected output? 
B must take part in the Conference

What do you see instead?
B takes part in the Conference but he also sends a REFER to the CF(packet 942). 
This REFER must not be sent

What version of the product are you using? On what operating system?
RI 2.5.11
Android 2.2

Please provide any additional information below.
A pcap file is attached.

Original issue reported on code.google.com by rcs.capg...@gmail.com on 28 Oct 2013 at 11:53

Attachments:

GoogleCodeExporter commented 9 years ago
Thanks for the information.
We already works on it.

The stack encounters this issue because the INVITE (caused by add participant) 
doesn't contains the list of participants unlike a INVITE caused by a new group 
chat. I don't know if this behavior is correct ?

The stack sends REFER to invite missing participants of a restarted group chat. 
As it doesn't have the list at this time, it sends REFER with all known 
participants...
To correct, we will move this feature at the NOTIFY reception to be sure to 
have a complete list of participants.

Original comment by benoit.j...@gmail.com on 28 Oct 2013 at 12:57

GoogleCodeExporter commented 9 years ago

Original comment by benoit.j...@gmail.com on 28 Oct 2013 at 12:57

GoogleCodeExporter commented 9 years ago
Yes, it is preferable to check if there is missing participants based on NOTIFY 
requests.

This mechanism must be done in case of a restarted group chat. 

But I think it should not be applied to this use case because the client left 
voluntarily the conference. 
Why did the client kept a list of participants for this conference?

Original comment by rcs.capg...@gmail.com on 28 Oct 2013 at 2:26

GoogleCodeExporter commented 9 years ago
Same as 173.
Delivered in V2.5.13

Original comment by benoit.j...@gmail.com on 12 Nov 2013 at 10:16