aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDaniel Gultsch <daniel.gultsch@rwth-aachen.de>2014-04-05 12:08:35 +0200
committerDaniel Gultsch <daniel.gultsch@rwth-aachen.de>2014-04-05 12:08:35 +0200
commit486f198632a81fb9c3e1a41bcc0a68fa86888fa8 (patch)
treedc77ec412c62a7ece04f0bdb69a68f7d1563d288
parent83a23ad3b1d06155e5861c07762ad51163fda318 (diff)
use proper, advertised sm namespace
-rw-r--r--src/eu/siacs/conversations/xmpp/XmppConnection.java27
-rw-r--r--src/eu/siacs/conversations/xmpp/stanzas/streammgmt/AckPacket.java4
-rw-r--r--src/eu/siacs/conversations/xmpp/stanzas/streammgmt/EnablePacket.java10
-rw-r--r--src/eu/siacs/conversations/xmpp/stanzas/streammgmt/RequestPacket.java4
-rw-r--r--src/eu/siacs/conversations/xmpp/stanzas/streammgmt/ResumePacket.java4
5 files changed, 24 insertions, 25 deletions
diff --git a/src/eu/siacs/conversations/xmpp/XmppConnection.java b/src/eu/siacs/conversations/xmpp/XmppConnection.java
index 114f0c2fd..1ca7cd8b1 100644
--- a/src/eu/siacs/conversations/xmpp/XmppConnection.java
+++ b/src/eu/siacs/conversations/xmpp/XmppConnection.java
@@ -74,6 +74,7 @@ public class XmppConnection implements Runnable {
private List<String> discoItems = new ArrayList<String>();
private String streamId = null;
+ private int smVersion = 3;
private int stanzasReceived = 0;
private int stanzasSent = 0;
@@ -241,13 +242,13 @@ public class XmppConnection implements Runnable {
Element enabled = tagReader.readElement(nextTag);
if ("true".equals(enabled.getAttribute("resume"))) {
this.streamId = enabled.getAttribute("id");
- Log.d(LOGTAG,account.getJid()+": stream managment enabled (resumable)");
+ Log.d(LOGTAG,account.getJid()+": stream managment("+smVersion+") enabled (resumable)");
} else {
- Log.d(LOGTAG,account.getJid()+": stream managment enabled");
+ Log.d(LOGTAG,account.getJid()+": stream managment("+smVersion+") enabled");
}
this.lastSessionStarted = SystemClock.elapsedRealtime();
this.stanzasReceived = 0;
- RequestPacket r = new RequestPacket();
+ RequestPacket r = new RequestPacket(smVersion);
tagWriter.writeStanzaAsync(r);
} else if (nextTag.isStart("resumed")) {
tagReader.readElement(nextTag);
@@ -256,7 +257,7 @@ public class XmppConnection implements Runnable {
Log.d(LOGTAG,account.getJid()+": session resumed");
} else if (nextTag.isStart("r")) {
tagReader.readElement(nextTag);
- AckPacket ack = new AckPacket(this.stanzasReceived);
+ AckPacket ack = new AckPacket(this.stanzasReceived,smVersion);
//Log.d(LOGTAG,ack.toString());
tagWriter.writeStanzaAsync(ack);
} else if (nextTag.isStart("a")) {
@@ -531,8 +532,8 @@ public class XmppConnection implements Runnable {
} else if (mechanisms.contains("DIGEST-MD5")) {
sendSaslAuthDigestMd5();
}
- } else if (this.streamFeatures.hasChild("sm") && streamId != null) {
- ResumePacket resume = new ResumePacket(this.streamId,stanzasReceived);
+ } else if (this.streamFeatures.hasChild("sm","urn:xmpp:sm:"+smVersion) && streamId != null) {
+ ResumePacket resume = new ResumePacket(this.streamId,stanzasReceived,smVersion);
this.tagWriter.writeStanzaAsync(resume);
} else if (this.streamFeatures.hasChild("bind") && shouldBind) {
sendBindRequest();
@@ -633,9 +634,13 @@ public class XmppConnection implements Runnable {
String resource = packet.findChild("bind").findChild("jid")
.getContent().split("/")[1];
account.setResource(resource);
- if (streamFeatures.hasChild("sm")) {
- String xmlns = streamFeatures.findChild("sm").getAttribute("xmlns");
- EnablePacket enable = new EnablePacket(xmlns);
+ if (streamFeatures.hasChild("sm","urn:xmpp:sm:3")) {
+ smVersion = 3;
+ EnablePacket enable = new EnablePacket(smVersion);
+ tagWriter.writeStanzaAsync(enable);
+ } else if (streamFeatures.hasChild("sm","urn:xmpp:sm:2")) {
+ smVersion = 2;
+ EnablePacket enable = new EnablePacket(smVersion);
tagWriter.writeStanzaAsync(enable);
}
sendInitialPresence();
@@ -765,7 +770,7 @@ public class XmppConnection implements Runnable {
public void sendPing() {
if (streamFeatures.hasChild("sm")) {
- tagWriter.writeStanzaAsync(new RequestPacket());
+ tagWriter.writeStanzaAsync(new RequestPacket(smVersion));
} else {
IqPacket iq = new IqPacket(IqPacket.TYPE_GET);
iq.setFrom(account.getFullJid());
@@ -849,7 +854,7 @@ public class XmppConnection implements Runnable {
}
public void r() {
- this.tagWriter.writeStanzaAsync(new RequestPacket());
+ this.tagWriter.writeStanzaAsync(new RequestPacket(smVersion));
}
public int getReceivedStanzas() {
diff --git a/src/eu/siacs/conversations/xmpp/stanzas/streammgmt/AckPacket.java b/src/eu/siacs/conversations/xmpp/stanzas/streammgmt/AckPacket.java
index 7bc66e9b9..6fe3ea2bd 100644
--- a/src/eu/siacs/conversations/xmpp/stanzas/streammgmt/AckPacket.java
+++ b/src/eu/siacs/conversations/xmpp/stanzas/streammgmt/AckPacket.java
@@ -4,9 +4,9 @@ import eu.siacs.conversations.xmpp.stanzas.AbstractStanza;
public class AckPacket extends AbstractStanza {
- public AckPacket(int sequence) {
+ public AckPacket(int sequence, int smVersion) {
super("a");
- this.setAttribute("xmlns","urn:xmpp:sm:3");
+ this.setAttribute("xmlns","urn:xmpp:sm:"+smVersion);
this.setAttribute("h", ""+sequence);
}
diff --git a/src/eu/siacs/conversations/xmpp/stanzas/streammgmt/EnablePacket.java b/src/eu/siacs/conversations/xmpp/stanzas/streammgmt/EnablePacket.java
index f6a0adb2d..0ca7a4fd1 100644
--- a/src/eu/siacs/conversations/xmpp/stanzas/streammgmt/EnablePacket.java
+++ b/src/eu/siacs/conversations/xmpp/stanzas/streammgmt/EnablePacket.java
@@ -4,15 +4,9 @@ import eu.siacs.conversations.xmpp.stanzas.AbstractStanza;
public class EnablePacket extends AbstractStanza {
- public EnablePacket() {
+ public EnablePacket(int smVersion) {
super("enable");
- this.setAttribute("xmlns","urn:xmpp:sm:3");
- this.setAttribute("resume", "true");
- }
-
- public EnablePacket(String xmlns) {
- super("enable");
- this.setAttribute("xmlns",xmlns);
+ this.setAttribute("xmlns","urn:xmpp:sm:"+smVersion);
this.setAttribute("resume", "true");
}
diff --git a/src/eu/siacs/conversations/xmpp/stanzas/streammgmt/RequestPacket.java b/src/eu/siacs/conversations/xmpp/stanzas/streammgmt/RequestPacket.java
index 4d409fd28..d42ae9b01 100644
--- a/src/eu/siacs/conversations/xmpp/stanzas/streammgmt/RequestPacket.java
+++ b/src/eu/siacs/conversations/xmpp/stanzas/streammgmt/RequestPacket.java
@@ -4,9 +4,9 @@ import eu.siacs.conversations.xmpp.stanzas.AbstractStanza;
public class RequestPacket extends AbstractStanza {
- public RequestPacket() {
+ public RequestPacket(int smVersion) {
super("r");
- this.setAttribute("xmlns","urn:xmpp:sm:3");
+ this.setAttribute("xmlns","urn:xmpp:sm:"+smVersion);
}
}
diff --git a/src/eu/siacs/conversations/xmpp/stanzas/streammgmt/ResumePacket.java b/src/eu/siacs/conversations/xmpp/stanzas/streammgmt/ResumePacket.java
index bebfa1a2c..dcf321013 100644
--- a/src/eu/siacs/conversations/xmpp/stanzas/streammgmt/ResumePacket.java
+++ b/src/eu/siacs/conversations/xmpp/stanzas/streammgmt/ResumePacket.java
@@ -4,9 +4,9 @@ import eu.siacs.conversations.xmpp.stanzas.AbstractStanza;
public class ResumePacket extends AbstractStanza {
- public ResumePacket(String id, int sequence) {
+ public ResumePacket(String id, int sequence, int smVersion) {
super("resume");
- this.setAttribute("xmlns","urn:xmpp:sm:3");
+ this.setAttribute("xmlns","urn:xmpp:sm:"+smVersion);
this.setAttribute("previd", id);
this.setAttribute("h", ""+sequence);
}