MUC service returns wrong number of occupants and duplicate occupants in service discovery

Description

Sending this to a MUC chat room:

<iq id="1" to="open_chat@conference.igniterealtime.org" type="get"> <query xmlns="http://jabber.org/protocol/disco#items"/> </iq>

Result:

<iq from='open_chat@conference.igniterealtime.org' to='myJid' type='result' id='1'> <query xmlns='http://jabber.org/protocol/disco#items'> <item jid='open_chat@conference.igniterealtime.org/aRyo'/> <item jid='open_chat@conference.igniterealtime.org/BigD'/> <item jid='open_chat@conference.igniterealtime.org/Tobias'/> <item jid='open_chat@conference.igniterealtime.org/csh'/> <item jid='open_chat@conference.igniterealtime.org/laurentb'/> <item jid='open_chat@conference.igniterealtime.org/Tobias'/> <item jid='open_chat@conference.igniterealtime.org/akrherz'/> <item jid='open_chat@conference.igniterealtime.org/Tobias'/> </query> </iq>

As you can see "Tobias" is listed 3 times, which seems wrong for me.

Might be related to this recent improvement: https://igniterealtime.atlassian.net/browse/OF-103#icft=OF-103

This also affects the "muc#roominfo_occupants" field, which returns a higher number of occupants, then it should.

Environment

None

Activity

Show:

Tom Evans January 7, 2016 at 8:13 PM

Submitted PR #498

Tom Evans January 7, 2016 at 6:24 PM

Thanks for the fresh repro/validation. I'll take a look at this one.

Tim Durden January 7, 2016 at 6:11 PM

Confirmed as still an issue in Openfire 4.0.0 beta (nightly: 5th Jan).

Replication Steps:

  • On a single Openfire instance, create a MUC room.

  • Connect to the MUC room from two different clients, but using the same nick.

  • Send the iq message as above:

    <iq id="1" to="ROOM@HOSTNAME" type="get"> <query xmlns="http://jabber.org/protocol/disco#items"/> </iq>

Observe, multiple results for the same user are returned.

csh November 2, 2015 at 6:47 PM

I can't remember anybody fixed it.

Daryl Herzmann November 2, 2015 at 5:17 PM

Anybody wish to comment on if this is still relevant?

Fixed

Details

Assignee

Reporter

Components

Fix versions

Affects versions

Priority

Created June 8, 2014 at 1:41 PM
Updated January 7, 2016 at 8:13 PM
Resolved January 7, 2016 at 8:13 PM

Flag notifications