Skip to content

open/close instead of stream:stream#117

Open
Pitel wants to merge 2 commits into
xmppo:masterfrom
Pitel:patch-1
Open

open/close instead of stream:stream#117
Pitel wants to merge 2 commits into
xmppo:masterfrom
Pitel:patch-1

Conversation

@Pitel

@Pitel Pitel commented Sep 11, 2014

Copy link
Copy Markdown

This change makes it compatible with latest draft (in fact, this has been there since revison 01), where <stream:stream> and </stream:stream> has been replaced with <open/> and <close/>.

This makes it work with latest stanza.io, but probably breaks clients still relying on old <stream:stream> method.

@Pitel

Pitel commented Dec 11, 2014

Copy link
Copy Markdown
Author

Ok, I've update the readme. Is this enough, or do you want more?

@Gordin

Gordin commented Feb 2, 2015

Copy link
Copy Markdown

Can this be merged? This would make node-xmpp-bosh work with the letast versions of stanza.io and latest HEAD of strophe.js (I want to make a new strophe.js release when this is merged) without workarounds.
Also, releasing a new version with this would be nice for packaging.

@kochka

kochka commented Feb 24, 2015

Copy link
Copy Markdown

Yes, a merge would be cool 👍

@ffPjrZUGXfcxuAj

Copy link
Copy Markdown

It's possible not break compatibility with old stream:stream client?

@Gordin

Gordin commented May 6, 2015

Copy link
Copy Markdown

@Camaran No, the new protocol is not backwards-compatible. The only thing you could do is implement both protocols, try the current one and if authentication fails try the deprecated protocol. Ideally we should just drop the deprecated protocol altogether though, all servers support the current protocol by now, node-xmpp-bosh is one of the last XMPP Servers/Proxies that still uses the deprecated protocol.
@dhruvbird Is there still a reason why you are not merging this?

@ffPjrZUGXfcxuAj

Copy link
Copy Markdown

@Gordin for example Jappix client not support the new protocol

@Gordin

Gordin commented May 6, 2015

Copy link
Copy Markdown

@Camaran Someone should file a bug with Jappix then, the old protocol is deprecated since February 2014 now… (Plus the changes on the client side are really not that big, they're just not compatible)

@BenV BenV mentioned this pull request Jun 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants