aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorhrxi <hrrrxi@gmail.com>2019-08-26 17:30:47 +0200
committerhrxi <hrrrxi@gmail.com>2019-08-27 12:14:53 +0200
commit6028fd15a81a084b63311bc61f7b48d9f3d00746 (patch)
treedbf26f0d8f1103477f79519eda97f521d9cb4eff
parent6083f446b47e258e0381c0c22755dbfa881c7df7 (diff)
downloaddino-6028fd15a81a084b63311bc61f7b48d9f3d00746.tar.gz
dino-6028fd15a81a084b63311bc61f7b48d9f3d00746.zip
Don't error on Jingle file transfer hash session-info
-rw-r--r--xmpp-vala/src/module/xep/0166_jingle.vala34
-rw-r--r--xmpp-vala/src/module/xep/0234_jingle_file_transfer.vala13
-rw-r--r--xmpp-vala/src/module/xep/0260_jingle_socks5_bytestreams.vala2
3 files changed, 44 insertions, 5 deletions
diff --git a/xmpp-vala/src/module/xep/0166_jingle.vala b/xmpp-vala/src/module/xep/0166_jingle.vala
index d6dbcd9e..06e3d5c8 100644
--- a/xmpp-vala/src/module/xep/0166_jingle.vala
+++ b/xmpp-vala/src/module/xep/0166_jingle.vala
@@ -47,12 +47,16 @@ public errordomain Error {
TRANSPORT_ERROR,
}
-StanzaNode? get_single_node_anyns(StanzaNode parent, string node_name) throws IqError {
+StanzaNode? get_single_node_anyns(StanzaNode parent, string? node_name = null) throws IqError {
StanzaNode? result = null;
foreach (StanzaNode child in parent.get_all_subnodes()) {
- if (child.name == node_name) {
+ if (node_name == null || child.name == node_name) {
if (result != null) {
- throw new IqError.BAD_REQUEST(@"multiple $(node_name) nodes");
+ if (node_name != null) {
+ throw new IqError.BAD_REQUEST(@"multiple $(node_name) nodes");
+ } else {
+ throw new IqError.BAD_REQUEST(@"expected single subnode");
+ }
}
result = child;
}
@@ -364,6 +368,7 @@ public interface ContentType : Object {
public abstract string content_type_ns_uri();
public abstract TransportType content_type_transport_type();
public abstract ContentParameters parse_content_parameters(StanzaNode description) throws IqError;
+ public abstract void handle_content_session_info(XmppStream stream, Session session, StanzaNode info, Iq.Stanza iq) throws IqError;
}
public interface ContentParameters : Object {
@@ -445,25 +450,28 @@ public class Session {
// Validate action.
switch (action) {
case "session-accept":
+ case "session-info":
case "session-terminate":
case "transport-accept":
+ case "transport-info":
case "transport-reject":
case "transport-replace":
- case "transport-info":
break;
case "content-accept":
case "content-add":
case "content-modify":
case "content-reject":
case "content-remove":
+ case "description-info":
case "security-info":
+ throw new IqError.NOT_IMPLEMENTED(@"$(action) is not implemented");
default:
throw new IqError.BAD_REQUEST("invalid action");
}
ContentNode? content = null;
StanzaNode? transport = null;
// Do some pre-processing.
- if (action != "session-terminate") {
+ if (action != "session-info" && action != "session-terminate") {
content = get_single_content_node(jingle);
verify_content(content);
switch (action) {
@@ -496,6 +504,9 @@ public class Session {
}
handle_session_accept(stream, content, jingle, iq);
break;
+ case "session-info":
+ handle_session_info(stream, jingle, iq);
+ break;
case "session-terminate":
handle_session_terminate(stream, jingle, iq);
break;
@@ -563,6 +574,19 @@ public class Session {
stream.get_module(Iq.Module.IDENTITY).send_iq(stream, new Iq.Stanza.result(iq));
// TODO(hrxi): also handle presence type=unavailable
}
+ void handle_session_info(XmppStream stream, StanzaNode jingle, Iq.Stanza iq) throws IqError {
+ StanzaNode? info = get_single_node_anyns(jingle);
+ if (info == null) {
+ // Jingle session ping
+ stream.get_module(Iq.Module.IDENTITY).send_iq(stream, new Iq.Stanza.result(iq));
+ return;
+ }
+ ContentType? content_type = stream.get_module(Module.IDENTITY).get_content_type(info.ns_uri);
+ if (content_type == null) {
+ throw new IqError.UNSUPPORTED_INFO("unknown session-info namespace");
+ }
+ content_type.handle_content_session_info(stream, this, info, iq);
+ }
void select_new_transport(XmppStream stream) {
Transport? new_transport = stream.get_module(Module.IDENTITY).select_transport(stream, type_, peer_full_jid, tried_transport_methods);
if (new_transport == null) {
diff --git a/xmpp-vala/src/module/xep/0234_jingle_file_transfer.vala b/xmpp-vala/src/module/xep/0234_jingle_file_transfer.vala
index 43c212f5..25fe3ce4 100644
--- a/xmpp-vala/src/module/xep/0234_jingle_file_transfer.vala
+++ b/xmpp-vala/src/module/xep/0234_jingle_file_transfer.vala
@@ -24,6 +24,19 @@ public class Module : Jingle.ContentType, XmppStreamModule {
public Jingle.ContentParameters parse_content_parameters(StanzaNode description) throws Jingle.IqError {
return Parameters.parse(this, description);
}
+ public void handle_content_session_info(XmppStream stream, Jingle.Session session, StanzaNode info, Iq.Stanza iq) throws Jingle.IqError {
+ switch (info.name) {
+ case "received":
+ stream.get_module(Iq.Module.IDENTITY).send_iq(stream, new Iq.Stanza.result(iq));
+ break;
+ case "checksum":
+ // TODO(hrxi): handle hash
+ stream.get_module(Iq.Module.IDENTITY).send_iq(stream, new Iq.Stanza.result(iq));
+ break;
+ default:
+ throw new Jingle.IqError.UNSUPPORTED_INFO(@"unsupported file transfer info $(info.name)");
+ }
+ }
public signal void file_incoming(XmppStream stream, FileTransfer file_transfer);
diff --git a/xmpp-vala/src/module/xep/0260_jingle_socks5_bytestreams.vala b/xmpp-vala/src/module/xep/0260_jingle_socks5_bytestreams.vala
index c17dc0b3..e0542207 100644
--- a/xmpp-vala/src/module/xep/0260_jingle_socks5_bytestreams.vala
+++ b/xmpp-vala/src/module/xep/0260_jingle_socks5_bytestreams.vala
@@ -477,6 +477,8 @@ class Parameters : Jingle.TransportParameters, Object {
throw new IOError.PROXY_FAILED("wanted port 00 00, got %02x %02x".printf(read_buffer[5+dstaddr.length], read_buffer[5+dstaddr.length+1]));
}
+ conn.get_socket().set_timeout(0);
+
return conn;
}
public async void try_connecting_to_candidates(XmppStream stream, Jingle.Session session) throws Error {