Use outgoing JMI if contact has supporting device

7 files changed, 99 insertions, 47 deletions

diff --git a/libdino/src/entity/call.vala b/libdino/src/entity/call.vala
index b836e2cd..7891dae7 100644
--- a/libdino/src/entity/call.vala
+++ b/libdino/src/entity/call.vala
@@ -77,6 +77,8 @@ namespace Dino.Entities {
                     .value(db.call.state, State.ENDED); // No point in persisting states that can't survive a restart
             if (end_time != null) {
                 builder.value(db.call.end_time, (long) end_time.to_unix());
+            } else {
+                builder.value(db.call.end_time, (long) local_time.to_unix());
             }
             id = (int) builder.perform();
 
diff --git a/libdino/src/service/calls.vala b/libdino/src/service/calls.vala
index 54c353b0..93636c03 100644
--- a/libdino/src/service/calls.vala
+++ b/libdino/src/service/calls.vala
@@ -21,15 +21,16 @@ namespace Dino {
         public string id { get { return IDENTITY.id; } }
 
         private StreamInteractor stream_interactor;
+        private Database db;
         private Xep.JingleRtp.SessionInfoType session_info_type;
 
         private HashMap<Account, HashMap<Call, string>> sid_by_call = new HashMap<Account, HashMap<Call, string>>(Account.hash_func, Account.equals_func);
         private HashMap<Account, HashMap<string, Call>> call_by_sid = new HashMap<Account, HashMap<string, Call>>(Account.hash_func, Account.equals_func);
         public HashMap<Call, Xep.Jingle.Session> sessions = new HashMap<Call, Xep.Jingle.Session>(Call.hash_func, Call.equals_func);
 
-        public Call? mi_accepted_call = null;
-        public string? mi_accepted_sid = null;
-        public bool mi_accepted_video = false;
+        public HashMap<Account, Call> jmi_call = new HashMap<Account, Call>(Account.hash_func, Account.equals_func);
+        public HashMap<Account, string> jmi_sid = new HashMap<Account, string>(Account.hash_func, Account.equals_func);
+        public HashMap<Account, bool> jmi_video = new HashMap<Account, bool>(Account.hash_func, Account.equals_func);
 
         private HashMap<Call, bool> counterpart_sends_video = new HashMap<Call, bool>(Call.hash_func, Call.equals_func);
         private HashMap<Call, bool> we_should_send_video = new HashMap<Call, bool>(Call.hash_func, Call.equals_func);
@@ -46,6 +47,7 @@ namespace Dino {
 
         private Calls(StreamInteractor stream_interactor, Database db) {
             this.stream_interactor = stream_interactor;
+            this.db = db;
 
             stream_interactor.account_added.connect(on_account_added);
         }
@@ -75,22 +77,32 @@ namespace Dino {
 
             stream_interactor.get_module(CallStore.IDENTITY).add_call(call, conversation);
 
-            XmppStream? stream = stream_interactor.get_stream(conversation.account);
-            if (stream == null) return null;
+            we_should_send_video[call] = video;
+            we_should_send_audio[call] = true;
 
-            Gee.List<Jid> call_resources = yield get_call_resources(conversation);
-            if (call_resources.size > 0) {
-                Jid full_jid = call_resources[0];
-                Xep.Jingle.Session session = yield stream.get_module(Xep.JingleRtp.Module.IDENTITY).start_call(stream, full_jid, video);
-                sessions[call] = session;
-                call_by_sid[call.account][session.sid] = call;
-                sid_by_call[call.account][call] = session.sid;
+            if (yield has_jmi_resources(conversation)) {
+                XmppStream? stream = stream_interactor.get_stream(conversation.account);
+                jmi_call[conversation.account] = call;
+                jmi_video[conversation.account] = video;
+                jmi_sid[conversation.account] = Xmpp.random_uuid();
 
-                connect_session_signals(call, session);
-            }
+                call_by_sid[call.account][jmi_sid[conversation.account]] = call;
 
-            we_should_send_video[call] = video;
-            we_should_send_audio[call] = true;
+                var descriptions = new ArrayList<StanzaNode>();
+                descriptions.add(new StanzaNode.build("description", "urn:xmpp:jingle:apps:rtp:1").add_self_xmlns().put_attribute("media", "audio"));
+                if (video) {
+                    descriptions.add(new StanzaNode.build("description", "urn:xmpp:jingle:apps:rtp:1").add_self_xmlns().put_attribute("media", "video"));
+                }
+
+                stream.get_module(Xmpp.Xep.JingleMessageInitiation.Module.IDENTITY).send_session_propose_to_peer(stream, conversation.counterpart, jmi_sid[call.account], descriptions);
+            } else {
+                Gee.List<Jid> call_resources = yield get_call_resources(conversation);
+                if (call_resources.size == 0) {
+                    warning("No call resources");
+                    return null;
+                }
+                yield call_resource(conversation.account, call_resources[0], call, video);
+            }
 
             conversation.last_active = call.time;
             call_outgoing(call, conversation);
@@ -98,6 +110,17 @@ namespace Dino {
             return call;
         }
 
+        private async void call_resource(Account account, Jid full_jid, Call call, bool video, string? sid = null) {
+            XmppStream? stream = stream_interactor.get_stream(account);
+            if (stream == null) return;
+
+            Xep.Jingle.Session session = yield stream.get_module(Xep.JingleRtp.Module.IDENTITY).start_call(stream, full_jid, video, sid);
+            sessions[call] = session;
+            sid_by_call[call.account][call] = session.sid;
+
+            connect_session_signals(call, session);
+        }
+
         public void end_call(Conversation conversation, Call call) {
             XmppStream? stream = stream_interactor.get_stream(call.account);
             if (stream == null) return;
@@ -130,15 +153,17 @@ namespace Dino {
                 }
             } else {
                 // Only a JMI so far
-                XmppStream stream = stream_interactor.get_stream(call.account);
+                Account account = call.account;
+                string sid = sid_by_call[call.account][call];
+                XmppStream stream = stream_interactor.get_stream(account);
                 if (stream == null) return;
 
-                mi_accepted_call = call;
-                mi_accepted_sid = sid_by_call[call.account][call];
-                mi_accepted_video = we_should_send_video[call];
+                jmi_call[account] = call;
+                jmi_sid[account] = sid;
+                jmi_video[account] = we_should_send_video[call];
 
-                stream.get_module(Xep.JingleMessageInitiation.Module.IDENTITY).send_session_accept_to_self(stream, mi_accepted_sid);
-                stream.get_module(Xep.JingleMessageInitiation.Module.IDENTITY).send_session_proceed_to_peer(stream, call.counterpart, mi_accepted_sid);
+                stream.get_module(Xep.JingleMessageInitiation.Module.IDENTITY).send_session_accept_to_self(stream, sid);
+                stream.get_module(Xep.JingleMessageInitiation.Module.IDENTITY).send_session_proceed_to_peer(stream, call.counterpart, sid);
             }
         }
 
@@ -211,7 +236,11 @@ namespace Dino {
             // If video_feed == null && !mute we're trying to mute a non-existant feed. It will be muted as soon as it is created.
         }
 
-        public async Gee.List<Jid> get_call_resources(Conversation conversation) {
+        public async bool can_do_calls(Conversation conversation) {
+            return (yield get_call_resources(conversation)).size > 0 || yield has_jmi_resources(conversation);
+        }
+
+        private async Gee.List<Jid> get_call_resources(Conversation conversation) {
             ArrayList<Jid> ret = new ArrayList<Jid>();
 
             XmppStream? stream = stream_interactor.get_stream(conversation.account);
@@ -228,6 +257,15 @@ namespace Dino {
             return ret;
         }
 
+        private async bool has_jmi_resources(Conversation conversation) {
+            int64 jmi_resources = db.entity.select()
+                    .with(db.entity.jid_id, "=", db.get_jid_id(conversation.counterpart))
+                    .join_with(db.entity_feature, db.entity.caps_hash, db.entity_feature.entity)
+                    .with(db.entity_feature.feature, "=", Xep.JingleMessageInitiation.NS_URI)
+                    .count();
+            return jmi_resources > 0;
+        }
+
         public bool should_we_send_video(Call call) {
             return we_should_send_video[call];
         }
@@ -252,13 +290,14 @@ namespace Dino {
             }
 
             // Session might have already been accepted via Jingle Message Initiation
-            bool already_accepted = mi_accepted_sid == session.sid && mi_accepted_call.account.equals(account) &&
-                    mi_accepted_call.counterpart.equals_bare(session.peer_full_jid) &&
-                    mi_accepted_video == counterpart_wants_video;
+            bool already_accepted = jmi_sid.contains(account) &&
+                    jmi_sid[account] == session.sid && jmi_call[account].account.equals(account) &&
+                    jmi_call[account].counterpart.equals_bare(session.peer_full_jid) &&
+                    jmi_video[account] == counterpart_wants_video;
 
             Call? call = null;
             if (already_accepted) {
-                call = mi_accepted_call;
+                call = jmi_call[account];
             } else {
                 call = create_received_call(account, session.peer_full_jid, account.full_jid, counterpart_wants_video);
             }
@@ -334,16 +373,15 @@ namespace Dino {
             }
             if (call.state == Call.State.IN_PROGRESS) {
                 call.state = Call.State.ENDED;
-                call_terminated(call, reason_name, reason_text);
             } else if (call.state == Call.State.RINGING || call.state == Call.State.ESTABLISHING) {
                 if (reason_name == Xep.Jingle.ReasonElement.DECLINE) {
                     call.state = Call.State.DECLINED;
                 } else {
                     call.state = Call.State.FAILED;
                 }
-                call_terminated(call, reason_name, reason_text);
             }
 
+            call_terminated(call, reason_name, reason_text);
             remove_call_from_datastructures(call);
         }
 
@@ -478,11 +516,21 @@ namespace Dino {
             mi_module.session_accepted.connect((from, sid) => {
                 if (!call_by_sid[account].has_key(sid)) return;
 
-                // Ignore session-accepted from ourselves
-                if (!from.equals(account.full_jid)) {
+                if (from.equals_bare(account.bare_jid)) { // Carboned message from our account
+                    // Ignore carbon from ourselves
+                    if (from.equals(account.full_jid)) return;
+
                     Call call = call_by_sid[account][sid];
                     call.state = Call.State.OTHER_DEVICE_ACCEPTED;
                     remove_call_from_datastructures(call);
+                } else if (from.equals_bare(call_by_sid[account][sid].counterpart)) { // Message from our peer
+                    // We proposed the call
+                    if (jmi_sid.has_key(account) && jmi_sid[account] == sid) {
+                        call_resource(account, from, jmi_call[account], jmi_video[account], jmi_sid[account]);
+                        jmi_call.unset(account);
+                        jmi_sid.unset(account);
+                        jmi_video.unset(account);
+                    }
                 }
             });
             mi_module.session_rejected.connect((from, to, sid) => {
diff --git a/main/src/ui/conversation_titlebar/call_entry.vala b/main/src/ui/conversation_titlebar/call_entry.vala
index 1ac4dd83..5e28ecbe 100644
--- a/main/src/ui/conversation_titlebar/call_entry.vala
+++ b/main/src/ui/conversation_titlebar/call_entry.vala
@@ -116,9 +116,9 @@ namespace Dino.Ui {
         private async void update_visibility() {
             if (conversation.type_ == Conversation.Type.CHAT) {
                 Conversation conv_bak = conversation;
-                Gee.List<Jid>? resources = yield stream_interactor.get_module(Calls.IDENTITY).get_call_resources(conversation);
+                bool can_do_calls = yield stream_interactor.get_module(Calls.IDENTITY).can_do_calls(conversation);
                 if (conv_bak != conversation) return;
-                visible = resources != null && resources.size > 0;
+                visible = can_do_calls;
             } else {
                 visible = false;
             }
diff --git a/plugins/ice/src/dtls_srtp.vala b/plugins/ice/src/dtls_srtp.vala
index b742ccab..f294e66b 100644
--- a/plugins/ice/src/dtls_srtp.vala
+++ b/plugins/ice/src/dtls_srtp.vala
@@ -249,12 +249,4 @@ public class DtlsSrtp {
         }
         return sb.str;
     }
-
-    private uint8[] uint8_pt_to_a(uint8* data, uint size) {
-        uint8[size] ret = new uint8[size];
-        for (int i = 0; i < size; i++) {
-            ret[i] = data[i];
-        }
-        return ret;
-    }
 }
\ No newline at end of file
diff --git a/xmpp-vala/src/module/xep/0166_jingle/jingle_module.vala b/xmpp-vala/src/module/xep/0166_jingle/jingle_module.vala
index 1e8a36d1..7314ca6c 100644
--- a/xmpp-vala/src/module/xep/0166_jingle/jingle_module.vala
+++ b/xmpp-vala/src/module/xep/0166_jingle/jingle_module.vala
@@ -102,7 +102,7 @@ namespace Xmpp.Xep.Jingle {
             return (yield is_jingle_available(stream, full_jid)) && (yield select_transport(stream, type, components, full_jid, Set.empty())) != null;
         }
 
-        public async Session create_session(XmppStream stream, Gee.List<Content> contents, Jid receiver_full_jid, string sid = random_uuid()) throws Error {
+        public async Session create_session(XmppStream stream, Gee.List<Content> contents, Jid receiver_full_jid, string? sid = null) throws Error {
             if (!yield is_jingle_available(stream, receiver_full_jid)) {
                 throw new Error.NO_SHARED_PROTOCOLS("No Jingle support");
             }
@@ -111,7 +111,7 @@ namespace Xmpp.Xep.Jingle {
                 throw new Error.GENERAL("Couldn't determine own JID");
             }
 
-            Session session = new Session.initiate_sent(stream, sid, my_jid, receiver_full_jid);
+            Session session = new Session.initiate_sent(stream, sid ?? random_uuid(), my_jid, receiver_full_jid);
             session.terminated.connect((session, stream, _1, _2, _3) => { stream.get_flag(Flag.IDENTITY).remove_session(session.sid); });
 
             foreach (Content content in contents) {
diff --git a/xmpp-vala/src/module/xep/0167_jingle_rtp/jingle_rtp_module.vala b/xmpp-vala/src/module/xep/0167_jingle_rtp/jingle_rtp_module.vala
index 3a9ea09f..3adad114 100644
--- a/xmpp-vala/src/module/xep/0167_jingle_rtp/jingle_rtp_module.vala
+++ b/xmpp-vala/src/module/xep/0167_jingle_rtp/jingle_rtp_module.vala
@@ -26,7 +26,7 @@ public abstract class Module : XmppStreamModule {
     public abstract Stream create_stream(Jingle.Content content);
     public abstract void close_stream(Stream stream);
 
-    public async Jingle.Session start_call(XmppStream stream, Jid receiver_full_jid, bool video) throws Jingle.Error {
+    public async Jingle.Session start_call(XmppStream stream, Jid receiver_full_jid, bool video, string? sid = null) throws Jingle.Error {
 
         Jingle.Module jingle_module = stream.get_module(Jingle.Module.IDENTITY);
 
@@ -72,7 +72,7 @@ public abstract class Module : XmppStreamModule {
 
         // Create session
         try {
-            Jingle.Session session = yield jingle_module.create_session(stream, contents, receiver_full_jid);
+            Jingle.Session session = yield jingle_module.create_session(stream, contents, receiver_full_jid, sid);
             return session;
         } catch (Jingle.Error e) {
             throw new Jingle.Error.GENERAL(@"Couldn't create Jingle session: $(e.message)");
diff --git a/xmpp-vala/src/module/xep/0353_jingle_message_initiation.vala b/xmpp-vala/src/module/xep/0353_jingle_message_initiation.vala
index acb2ba2e..dbb6fd81 100644
--- a/xmpp-vala/src/module/xep/0353_jingle_message_initiation.vala
+++ b/xmpp-vala/src/module/xep/0353_jingle_message_initiation.vala
@@ -1,7 +1,7 @@
 using Gee;
 
 namespace Xmpp.Xep.JingleMessageInitiation {
-    private const string NS_URI = "urn:xmpp:jingle-message:0";
+    public const string NS_URI = "urn:xmpp:jingle-message:0";
 
     public class Module : XmppStreamModule {
         public static ModuleIdentity<Module> IDENTITY = new ModuleIdentity<Module>(NS_URI, "0353_jingle_message_initiation");
@@ -11,6 +11,17 @@ namespace Xmpp.Xep.JingleMessageInitiation {
         public signal void session_accepted(Jid from, string sid);
         public signal void session_rejected(Jid from, Jid to, string sid);
 
+        public void send_session_propose_to_peer(XmppStream stream, Jid to, string sid, Gee.List<StanzaNode> descriptions) {
+            StanzaNode propose_node = new StanzaNode.build("propose", NS_URI).add_self_xmlns().put_attribute("id", sid, NS_URI);
+            foreach (StanzaNode desc_node in descriptions) {
+                propose_node.put_node(desc_node);
+            }
+
+            MessageStanza accepted_message = new MessageStanza() { to=to };
+            accepted_message.stanza.put_node(propose_node);
+            stream.get_module(MessageModule.IDENTITY).send_message.begin(stream, accepted_message);
+        }
+
         public void send_session_accept_to_self(XmppStream stream, string sid) {
             MessageStanza accepted_message = new MessageStanza() { to=Bind.Flag.get_my_jid(stream).bare_jid };
             accepted_message.stanza.put_node(
@@ -58,7 +69,6 @@ namespace Xmpp.Xep.JingleMessageInitiation {
             switch (mi_node.name) {
                 case "accept":
                 case "proceed":
-                    if (!message.from.equals_bare(Bind.Flag.get_my_jid(stream))) return;
                     session_accepted(message.from, mi_node.get_attribute("id"));
                     break;
                 case "propose":