Skip to content

Resource tracking in OutQueueConsumer seems defunct #277

Open

Description

@ashward

The resource tracking code in OutQueueConsumer doesn't seem to work. Nothing seems to be actually tracking resources so the onlineUsers map never seems to get populated with anything. It's causing me a problem with notifications to local admin users.

I guess we could safely get rid of that code now - specifically this bit:

            // Get a list of 'online' resources for this JID
            ArrayList<JID> resources = onlineUsers.getResources(p.getTo());
            logger.debug("There are " + resources.size() + " online resources for " + p.getTo());
            for (JID resource : resources) {
                p.setTo(resource);
                component.sendPacket(p.createCopy());
            }

Happy to make the change myself, but thought I'd check there's no reason for it to still be in there.

Activity

  1. ashward commented on Nov 25, 2014

    @ashward
    ContributorAuthor

    Ah, you're right. But it doesn't seem to be working - it's coming up as 0 resources for all the local users as far as I can tell. I'll do a bit more digging when I have a bit more spare time :)

  2. lloydwatkin commented on Nov 25, 2014

    @lloydwatkin
    Member

    Are you using database or in-memory store? If database have you applied the most recent updates?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions