Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Get list of all clients in a room #79

Open
rmc3 opened this issue Apr 3, 2015 · 2 comments
Open

Get list of all clients in a room #79

rmc3 opened this issue Apr 3, 2015 · 2 comments

Comments

@rmc3
Copy link

rmc3 commented Apr 3, 2015

I'm having an issue that may be related to #41, but may also be expected behavior. I'm hoping we can confirm whether it's expected behavior or an issue in my environment. It's worth noting that OpenFire is the XMPP server implementation in use here.

When the bot joins a room, robot.brain.data.users is empty. After it sees a message from a user for the first time, an entry for that user is added. Also, clients never seem to receive a presence notification for the bot: at least on Adium, the bot never appears in the list of users in the room, though clients can receive messages sent by the bot to the room.

What I'd like to be able to do is send unsolicited messages generated by POSTs to Hubot's HTTP endpoints to individual users in the room. However, with the current behavior, the bot doesn't "know about" users in the room until they send a message in it.

Is this expected behavior? Are there any workarounds? It sounds like #65 could add functionality that would fix this issue.

@markstory
Copy link
Contributor

The bot should create user objects as it hears presence messages from each user in the channel. You should be able to see hubot-xmpp receiving those messages if you enable debug logging.

Alternatively, the only hold up on merging #65 was the lack of tests, if you could add those in I'd be happy to merge that set of changes in.

@rmc3
Copy link
Author

rmc3 commented Apr 7, 2015

Roger. I'll take a stab at writing the tests for it in the next few days, that sounds like exactly the functionality I'm looking for.

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

No branches or pull requests

2 participants