qxmpp
qxmpp copied to clipboard
get resume id from server
LGTM.
IQ parsing / serialization is trivial to unit test, so do it :)
@coderlxn can we please have a unit test for this so we can merge it?
@lnjX I'm not sure I understand what this does anymore, ideas?
@jlaine The session id needs to be parsed to be able to resume the session (using previd
in <resume/>
) after a possible connection loss.
See https://xmpp.org/extensions/xep-0198.html#resumption
@jlaine What kind of unit test should I add for this pull request? Sorry, I'm not familiar with the progress of pull request.
Unfortunately there's no test class for stream management, yet, but you could base on this test basically. It were also nice to have this new test class in a new commit.