xmpp-ftw / xmpp-ftw-muc

Multi User Chat/MUC (XEP-0045) plugin for XMPP-FTW
Apache License 2.0
5 stars 3 forks source link

"xmpp.muc.role.get" -> room dead #53

Open robotnic opened 9 years ago

robotnic commented 9 years ago

When I use the "xmpp.muc.role.get" stanza, I don't get an answer. The connection seams to be dead after that stanza.

3:::{"type":0,"data":["xmpp.muc.role.get",{"room":"testing2@channels.buddycloud.com","role":"participants"}],"id":8}

lloydwatkin commented 9 years ago

I'm unable to recreate this locally:

screenshot from 2015-01-19 17 29 58

Are you able to provide further details/logs?

robotnic commented 9 years ago

You can test here http://datenkueche.com/buddycloud/ (login -> groupchat -> type nickname -> press button at right bottom corner -> dead

Socket connections are going to "https://xmpp-ftw.jit.su/"

lloydwatkin commented 9 years ago

Afraid without seeing logs I can't really help. May I suggest you do the following:

npm i xmpp-ftw-demo
cd node_modules/xmpp-ftw-demo
node index

Point your websocket connector at http://localhost:3000. Let me know any errors that appear whilst repeating your steps from above.

robotnic commented 9 years ago

I installed that again on my computer. Same bug.

bildschirmfoto von 2015-01-19 21 39 35

After "xmpp.muc.role.get" no stanzas are received.

lloydwatkin commented 9 years ago

Can you run xmpp-ftw-demo with environment variable 'DEBUG' to '*', that will tell you everything that is going on. Please post your logs in a gist. On 19 Jan 2015 20:42, "bz" notifications@github.com wrote:

I installed that again on my computer. Same bug.

[image: bildschirmfoto von 2015-01-19 21 39 35] https://cloud.githubusercontent.com/assets/219020/5807456/d086c35e-a023-11e4-920d-c4c65a3ac13c.png

After "xmpp.muc.role.get" no stanzas are received.

— Reply to this email directly or view it on GitHub https://github.com/xmpp-ftw/xmpp-ftw-muc/issues/53#issuecomment-70557264 .

robotnic commented 9 years ago

[TypeError: Cannot read property 'NS_ADMIN' of undefined]

lloydwatkin commented 9 years ago

Could you try changing this line https://github.com/xmpp-ftw/xmpp-ftw-muc/blob/master/lib/multi-user-chat.js#L352 to read

var items = stanza.getChild('query', self.NS_ADMIN)
            .getChildren('item')

...and let me know if that corrects things? (I suspect it will)

robotnic commented 9 years ago

That works. I made a pull request https://github.com/xmpp-ftw/xmpp-ftw-muc/pull/54

lloydwatkin commented 9 years ago

Excellent, I'll get merged and released next time I am at a computer. On 20 Jan 2015 14:31, "bz" notifications@github.com wrote:

That works. I made a pull request

54 https://github.com/xmpp-ftw/xmpp-ftw-muc/pull/54

— Reply to this email directly or view it on GitHub https://github.com/xmpp-ftw/xmpp-ftw-muc/issues/53#issuecomment-70661424 .