From 7c4260eed718961874fc0ea665263ea2ce59338b Mon Sep 17 00:00:00 2001 From: fiaxh Date: Tue, 21 Apr 2020 16:25:21 +0200 Subject: Remove features from service discovery module when detaching module from stream fixes #179 fixes #812 --- .../module/xep/0030_service_discovery/flag.vala | 36 +++++++++++++++++++--- .../xep/0030_service_discovery/identity.vala | 12 ++++++++ .../xep/0030_service_discovery/info_result.vala | 4 +-- .../module/xep/0030_service_discovery/module.vala | 33 +++++++++++++++----- xmpp-vala/src/module/xep/0045_muc/module.vala | 5 ++- xmpp-vala/src/module/xep/0060_pubsub.vala | 6 ++++ xmpp-vala/src/module/xep/0084_user_avatars.vala | 4 ++- .../module/xep/0085_chat_state_notifications.vala | 1 + .../src/module/xep/0115_entitiy_capabilities.vala | 10 ++++-- xmpp-vala/src/module/xep/0166_jingle.vala | 1 + .../module/xep/0184_message_delivery_receipts.vala | 1 + .../src/module/xep/0191_blocking_command.vala | 3 +- xmpp-vala/src/module/xep/0199_ping.vala | 1 + .../src/module/xep/0234_jingle_file_transfer.vala | 4 ++- .../module/xep/0260_jingle_socks5_bytestreams.vala | 4 ++- .../xep/0261_jingle_in_band_bytestreams.vala | 4 ++- xmpp-vala/src/module/xep/0280_message_carbons.vala | 1 + .../module/xep/0308_last_message_correction.vala | 4 ++- xmpp-vala/src/module/xep/0333_chat_markers.vala | 3 +- .../module/xep/0359_unique_stable_stanza_ids.vala | 4 ++- .../xep/0391_jingle_encrypted_transports.vala | 3 +- xmpp-vala/src/module/xep/0402_bookmarks2.vala | 4 ++- 22 files changed, 117 insertions(+), 31 deletions(-) (limited to 'xmpp-vala/src/module/xep') diff --git a/xmpp-vala/src/module/xep/0030_service_discovery/flag.vala b/xmpp-vala/src/module/xep/0030_service_discovery/flag.vala index 4661fede..d4874fca 100644 --- a/xmpp-vala/src/module/xep/0030_service_discovery/flag.vala +++ b/xmpp-vala/src/module/xep/0030_service_discovery/flag.vala @@ -6,11 +6,24 @@ public class Flag : XmppStreamFlag { public static FlagIdentity IDENTITY = new FlagIdentity(NS_URI, "service_discovery"); private HashMap?> entity_features = new HashMap?>(Jid.hash_func, Jid.equals_func); - private HashMap?> entity_identities = new HashMap?>(Jid.hash_func, Jid.equals_func); + private HashMap?> entity_identities = new HashMap?>(Jid.hash_func, Jid.equals_func); private HashMap?> entity_items = new HashMap?>(Jid.hash_func, Jid.equals_func); - public Gee.List features = new ArrayList(); - public Gee.List? get_entity_categories(Jid jid) { + private Gee.Set own_features_ = new HashSet(); + public Gee.List own_features { + owned get { + var ret = new ArrayList(); + foreach (var feature in own_features_) ret.add(feature); + return ret; + } + } + + private Gee.Set own_identities_ = new HashSet(Identity.hash_func, Identity.equals_func); + public Gee.Set own_identities { + owned get { return own_identities_.read_only_view; } + } + + public Gee.Set? get_entity_categories(Jid jid) { return entity_identities.has_key(jid) ? entity_identities[jid] : null; // TODO isn’t this default for hashmap } @@ -29,7 +42,7 @@ public class Flag : XmppStreamFlag { return entity_features[jid].contains(feature); } - public void set_entity_identities(Jid jid, Gee.List? identities) { + public void set_entity_identities(Jid jid, Gee.Set? identities) { entity_identities[jid] = identities; } @@ -41,7 +54,20 @@ public class Flag : XmppStreamFlag { entity_items[jid] = features; } - public void add_own_feature(string feature) { features.add(feature); } + public void add_own_feature(string feature) { + if (own_features_.contains(feature)) { + warning("Tried to add the feature %s a second time".printf(feature)); + return; + } + own_features_.add(feature); + } + + public void remove_own_feature(string feature) { + own_features_.remove(feature); + } + + public void add_own_identity(Identity identity) { own_identities_.add(identity); } + public void remove_own_identity(Identity identity) { own_identities_.remove(identity); } public override string get_ns() { return NS_URI; } diff --git a/xmpp-vala/src/module/xep/0030_service_discovery/identity.vala b/xmpp-vala/src/module/xep/0030_service_discovery/identity.vala index 43aecfb2..aec2ed24 100644 --- a/xmpp-vala/src/module/xep/0030_service_discovery/identity.vala +++ b/xmpp-vala/src/module/xep/0030_service_discovery/identity.vala @@ -17,6 +17,18 @@ public class Identity { this.type_ = type; this.name = name; } + + public static uint hash_func(Identity a) { + uint hash = a.category.hash() ^ a.type_.hash(); + if (a.name != null) { + hash ^= a.name.hash(); + } + return hash; + } + + public static bool equals_func(Identity a, Identity b) { + return a.category == b.category && a.type_ == b.type_ && a.name == b.name; + } } } diff --git a/xmpp-vala/src/module/xep/0030_service_discovery/info_result.vala b/xmpp-vala/src/module/xep/0030_service_discovery/info_result.vala index 65252261..a9606115 100644 --- a/xmpp-vala/src/module/xep/0030_service_discovery/info_result.vala +++ b/xmpp-vala/src/module/xep/0030_service_discovery/info_result.vala @@ -20,9 +20,9 @@ public class InfoResult { } } - public Gee.List identities { + public Gee.Set identities { owned get { - ArrayList ret = new ArrayList(); + HashSet ret = new HashSet(); foreach (StanzaNode feature_node in iq.stanza.get_subnode("query", NS_URI_INFO).get_subnodes("identity", NS_URI_INFO)) { ret.add(new Identity(feature_node.get_attribute("category", NS_URI_INFO), feature_node.get_attribute("type", NS_URI_INFO), diff --git a/xmpp-vala/src/module/xep/0030_service_discovery/module.vala b/xmpp-vala/src/module/xep/0030_service_discovery/module.vala index e2bb46f8..332e8fc3 100644 --- a/xmpp-vala/src/module/xep/0030_service_discovery/module.vala +++ b/xmpp-vala/src/module/xep/0030_service_discovery/module.vala @@ -9,27 +9,39 @@ public const string NS_URI_ITEMS = NS_URI + "#items"; public class Module : XmppStreamModule, Iq.Handler { public static ModuleIdentity IDENTITY = new ModuleIdentity(NS_URI, "0030_service_discovery_module"); - public ArrayList identities = new ArrayList(); + public Identity own_identity; public Module.with_identity(string category, string type, string? name = null) { - add_identity(category, type, name); + own_identity = new Identity(category, type, name); } public void add_feature(XmppStream stream, string feature) { stream.get_flag(Flag.IDENTITY).add_own_feature(feature); } + public void remove_feature(XmppStream stream, string feature) { + stream.get_flag(Flag.IDENTITY).remove_own_feature(feature); + } + public void add_feature_notify(XmppStream stream, string feature) { add_feature(stream, feature + "+notify"); } - public void add_identity(string category, string type, string? name = null) { - identities.add(new Identity(category, type, name)); + public void remove_feature_notify(XmppStream stream, string feature) { + remove_feature(stream, feature + "+notify"); + } + + public void add_identity(XmppStream stream, Identity identity) { + stream.get_flag(Flag.IDENTITY).add_own_identity(identity); } - public delegate void HasEntryCategoryRes(XmppStream stream, Gee.List? identities); + public void remove_identity(XmppStream stream, Identity identity) { + stream.get_flag(Flag.IDENTITY).remove_own_identity(identity); + } + + public delegate void HasEntryCategoryRes(XmppStream stream, Gee.Set? identities); public void get_entity_categories(XmppStream stream, Jid jid, owned HasEntryCategoryRes listener) { - Gee.List? res = stream.get_flag(Flag.IDENTITY).get_entity_categories(jid); + Gee.Set? res = stream.get_flag(Flag.IDENTITY).get_entity_categories(jid); if (res != null) listener(stream, res); request_info(stream, jid, (stream, query_result) => { listener(stream, query_result != null ? query_result.identities : null); @@ -68,12 +80,17 @@ public class Module : XmppStreamModule, Iq.Handler { public override void attach(XmppStream stream) { stream.add_flag(new Flag()); + add_identity(stream, own_identity); + stream.get_module(Iq.Module.IDENTITY).register_for_namespace(NS_URI_INFO, this); add_feature(stream, NS_URI_INFO); } public override void detach(XmppStream stream) { + remove_identity(stream, own_identity); + stream.get_module(Iq.Module.IDENTITY).unregister_from_namespace(NS_URI_INFO, this); + remove_feature(stream, NS_URI_INFO); } public static void require(XmppStream stream) { @@ -85,8 +102,8 @@ public class Module : XmppStreamModule, Iq.Handler { private void send_query_result(XmppStream stream, Iq.Stanza iq_request) { InfoResult query_result = new ServiceDiscovery.InfoResult(iq_request); - query_result.features = stream.get_flag(Flag.IDENTITY).features; - query_result.identities = identities; + query_result.features = stream.get_flag(Flag.IDENTITY).own_features; + query_result.identities = stream.get_flag(Flag.IDENTITY).own_identities; stream.get_module(Iq.Module.IDENTITY).send_iq(stream, query_result.iq); } } diff --git a/xmpp-vala/src/module/xep/0045_muc/module.vala b/xmpp-vala/src/module/xep/0045_muc/module.vala index 263fa3d7..1b16f8a1 100644 --- a/xmpp-vala/src/module/xep/0045_muc/module.vala +++ b/xmpp-vala/src/module/xep/0045_muc/module.vala @@ -221,9 +221,7 @@ public class Module : XmppStreamModule { stream.get_module(Presence.Module.IDENTITY).received_presence.connect(check_for_enter_error); stream.get_module(Presence.Module.IDENTITY).received_available.connect(on_received_available); stream.get_module(Presence.Module.IDENTITY).received_unavailable.connect(on_received_unavailable); - if (stream.get_module(ServiceDiscovery.Module.IDENTITY) != null) { - stream.get_module(ServiceDiscovery.Module.IDENTITY).add_feature(stream, NS_URI); - } + stream.get_module(ServiceDiscovery.Module.IDENTITY).add_feature(stream, NS_URI); } public override void detach(XmppStream stream) { @@ -232,6 +230,7 @@ public class Module : XmppStreamModule { stream.get_module(Presence.Module.IDENTITY).received_presence.disconnect(check_for_enter_error); stream.get_module(Presence.Module.IDENTITY).received_available.disconnect(on_received_available); stream.get_module(Presence.Module.IDENTITY).received_unavailable.disconnect(on_received_unavailable); + stream.get_module(ServiceDiscovery.Module.IDENTITY).remove_feature(stream, NS_URI); } public override string get_ns() { return NS_URI; } diff --git a/xmpp-vala/src/module/xep/0060_pubsub.vala b/xmpp-vala/src/module/xep/0060_pubsub.vala index 279d87c3..9e5d8651 100644 --- a/xmpp-vala/src/module/xep/0060_pubsub.vala +++ b/xmpp-vala/src/module/xep/0060_pubsub.vala @@ -29,6 +29,12 @@ namespace Xmpp.Xep.Pubsub { } } + public void remove_filtered_notification(XmppStream stream, string node) { + stream.get_module(ServiceDiscovery.Module.IDENTITY).remove_feature_notify(stream, node); + item_listeners.unset(node); + retract_listeners.unset(node); + } + public async Gee.List? request_all(XmppStream stream, Jid jid, string node) { // TODO multiple nodes gehen auch Iq.Stanza request_iq = new Iq.Stanza.get(new StanzaNode.build("pubsub", NS_URI).add_self_xmlns().put_node(new StanzaNode.build("items", NS_URI).put_attribute("node", node))); request_iq.to = jid; diff --git a/xmpp-vala/src/module/xep/0084_user_avatars.vala b/xmpp-vala/src/module/xep/0084_user_avatars.vala index 9722eed5..9b21eeaf 100644 --- a/xmpp-vala/src/module/xep/0084_user_avatars.vala +++ b/xmpp-vala/src/module/xep/0084_user_avatars.vala @@ -35,7 +35,9 @@ namespace Xmpp.Xep.UserAvatars { stream.get_module(Pubsub.Module.IDENTITY).add_filtered_notification(stream, NS_URI_METADATA, on_pupsub_event, null); } - public override void detach(XmppStream stream) { } + public override void detach(XmppStream stream) { + stream.get_module(Pubsub.Module.IDENTITY).remove_filtered_notification(stream, NS_URI_METADATA); + } public void on_pupsub_event(XmppStream stream, Jid jid, string id, StanzaNode? node) { diff --git a/xmpp-vala/src/module/xep/0085_chat_state_notifications.vala b/xmpp-vala/src/module/xep/0085_chat_state_notifications.vala index bf7515cb..d8994551 100644 --- a/xmpp-vala/src/module/xep/0085_chat_state_notifications.vala +++ b/xmpp-vala/src/module/xep/0085_chat_state_notifications.vala @@ -37,6 +37,7 @@ public class Module : XmppStreamModule { } public override void detach(XmppStream stream) { + stream.get_module(ServiceDiscovery.Module.IDENTITY).remove_feature(stream, NS_URI); stream.get_module(MessageModule.IDENTITY).received_message.disconnect(on_received_message); stream.get_module(MessageModule.IDENTITY).send_pipeline.disconnect(send_pipeline_listener); } diff --git a/xmpp-vala/src/module/xep/0115_entitiy_capabilities.vala b/xmpp-vala/src/module/xep/0115_entitiy_capabilities.vala index 22946d4b..1d5d10c9 100644 --- a/xmpp-vala/src/module/xep/0115_entitiy_capabilities.vala +++ b/xmpp-vala/src/module/xep/0115_entitiy_capabilities.vala @@ -28,7 +28,7 @@ namespace Xmpp.Xep.EntityCapabilities { private string get_own_hash(XmppStream stream) { if (own_ver_hash == null) { - own_ver_hash = compute_hash(stream.get_module(ServiceDiscovery.Module.IDENTITY).identities, stream.get_flag(ServiceDiscovery.Flag.IDENTITY).features, new ArrayList()); + own_ver_hash = compute_hash(stream.get_flag(ServiceDiscovery.Flag.IDENTITY).own_identities, stream.get_flag(ServiceDiscovery.Flag.IDENTITY).own_features, new ArrayList()); } return own_ver_hash; } @@ -42,6 +42,7 @@ namespace Xmpp.Xep.EntityCapabilities { public override void detach(XmppStream stream) { stream.get_module(Presence.Module.IDENTITY).pre_send_presence_stanza.disconnect(on_pre_send_presence_stanza); stream.get_module(Presence.Module.IDENTITY).received_presence.disconnect(on_received_presence); + stream.get_module(ServiceDiscovery.Module.IDENTITY).remove_feature(stream, NS_URI); } public override string get_ns() { return NS_URI; } @@ -86,7 +87,10 @@ namespace Xmpp.Xep.EntityCapabilities { } } - private static string compute_hash(Gee.List identities, Gee.List features, Gee.List data_forms) { + private static string compute_hash(Gee.Set identities_set, Gee.List features, Gee.List data_forms) { + var identities = new ArrayList(); + foreach (var identity in identities_set) identities.add(identity); + identities.sort(compare_identities); features.sort(); @@ -154,7 +158,7 @@ namespace Xmpp.Xep.EntityCapabilities { } public interface Storage : Object { - public abstract void store_identities(string entity, Gee.List identities); + public abstract void store_identities(string entity, Gee.Set identities); public abstract void store_features(string entity, Gee.List capabilities); public abstract ServiceDiscovery.Identity? get_identities(string entity); public abstract Gee.List get_features(string entity); diff --git a/xmpp-vala/src/module/xep/0166_jingle.vala b/xmpp-vala/src/module/xep/0166_jingle.vala index ca368f00..7274c757 100644 --- a/xmpp-vala/src/module/xep/0166_jingle.vala +++ b/xmpp-vala/src/module/xep/0166_jingle.vala @@ -127,6 +127,7 @@ public class Module : XmppStreamModule, Iq.Handler { current_stream = stream; } public override void detach(XmppStream stream) { + stream.get_module(ServiceDiscovery.Module.IDENTITY).remove_feature(stream, NS_URI); stream.get_module(Iq.Module.IDENTITY).unregister_from_namespace(NS_URI, this); } diff --git a/xmpp-vala/src/module/xep/0184_message_delivery_receipts.vala b/xmpp-vala/src/module/xep/0184_message_delivery_receipts.vala index 199cfee8..7f4f8a03 100644 --- a/xmpp-vala/src/module/xep/0184_message_delivery_receipts.vala +++ b/xmpp-vala/src/module/xep/0184_message_delivery_receipts.vala @@ -26,6 +26,7 @@ namespace Xmpp.Xep.MessageDeliveryReceipts { } public override void detach(XmppStream stream) { + stream.get_module(ServiceDiscovery.Module.IDENTITY).remove_feature(stream, NS_URI); stream.get_module(MessageModule.IDENTITY).received_message.disconnect(received_message); stream.get_module(MessageModule.IDENTITY).send_pipeline.disconnect(send_pipeline_listener); } diff --git a/xmpp-vala/src/module/xep/0191_blocking_command.vala b/xmpp-vala/src/module/xep/0191_blocking_command.vala index ee8a1fed..2fc1d3b4 100644 --- a/xmpp-vala/src/module/xep/0191_blocking_command.vala +++ b/xmpp-vala/src/module/xep/0191_blocking_command.vala @@ -72,8 +72,9 @@ public class Module : XmppStreamModule, Iq.Handler { } public override void detach(XmppStream stream) { - stream.stream_negotiated.disconnect(on_stream_negotiated); stream.get_module(Iq.Module.IDENTITY).unregister_from_namespace(NS_URI, this); + stream.get_module(ServiceDiscovery.Module.IDENTITY).remove_feature(stream, NS_URI); + stream.stream_negotiated.disconnect(on_stream_negotiated); } public override string get_ns() { return NS_URI; } diff --git a/xmpp-vala/src/module/xep/0199_ping.vala b/xmpp-vala/src/module/xep/0199_ping.vala index 3458fa9d..07ee8ec8 100644 --- a/xmpp-vala/src/module/xep/0199_ping.vala +++ b/xmpp-vala/src/module/xep/0199_ping.vala @@ -22,6 +22,7 @@ namespace Xmpp.Xep.Ping { public override void detach(XmppStream stream) { stream.get_module(Iq.Module.IDENTITY).unregister_from_namespace(NS_URI, this); + stream.get_module(ServiceDiscovery.Module.IDENTITY).remove_feature(stream, NS_URI); } public void on_iq_get(XmppStream stream, Iq.Stanza iq) { 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 7780d973..d3a1f4bf 100644 --- a/xmpp-vala/src/module/xep/0234_jingle_file_transfer.vala +++ b/xmpp-vala/src/module/xep/0234_jingle_file_transfer.vala @@ -13,7 +13,9 @@ public class Module : Jingle.ContentType, XmppStreamModule { stream.get_module(ServiceDiscovery.Module.IDENTITY).add_feature(stream, NS_URI); stream.get_module(Jingle.Module.IDENTITY).register_content_type(this); } - public override void detach(XmppStream stream) { } + public override void detach(XmppStream stream) { + stream.get_module(ServiceDiscovery.Module.IDENTITY).remove_feature(stream, NS_URI); + } public string content_type_ns_uri() { return NS_URI; 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 c6556eb7..2ae2881e 100644 --- a/xmpp-vala/src/module/xep/0260_jingle_socks5_bytestreams.vala +++ b/xmpp-vala/src/module/xep/0260_jingle_socks5_bytestreams.vala @@ -13,7 +13,9 @@ public class Module : Jingle.Transport, XmppStreamModule { stream.get_module(Jingle.Module.IDENTITY).register_transport(this); stream.get_module(ServiceDiscovery.Module.IDENTITY).add_feature(stream, NS_URI); } - public override void detach(XmppStream stream) { } + public override void detach(XmppStream stream) { + stream.get_module(ServiceDiscovery.Module.IDENTITY).remove_feature(stream, NS_URI); + } public override string get_ns() { return NS_URI; } public override string get_id() { return IDENTITY.id; } diff --git a/xmpp-vala/src/module/xep/0261_jingle_in_band_bytestreams.vala b/xmpp-vala/src/module/xep/0261_jingle_in_band_bytestreams.vala index 1a810ee8..fc6bc057 100644 --- a/xmpp-vala/src/module/xep/0261_jingle_in_band_bytestreams.vala +++ b/xmpp-vala/src/module/xep/0261_jingle_in_band_bytestreams.vala @@ -14,7 +14,9 @@ public class Module : Jingle.Transport, XmppStreamModule { stream.get_module(Jingle.Module.IDENTITY).register_transport(this); stream.get_module(ServiceDiscovery.Module.IDENTITY).add_feature(stream, NS_URI); } - public override void detach(XmppStream stream) { } + public override void detach(XmppStream stream) { + stream.get_module(ServiceDiscovery.Module.IDENTITY).remove_feature(stream, NS_URI); + } public override string get_ns() { return NS_URI; } public override string get_id() { return IDENTITY.id; } diff --git a/xmpp-vala/src/module/xep/0280_message_carbons.vala b/xmpp-vala/src/module/xep/0280_message_carbons.vala index 508d3322..8d45ad38 100644 --- a/xmpp-vala/src/module/xep/0280_message_carbons.vala +++ b/xmpp-vala/src/module/xep/0280_message_carbons.vala @@ -26,6 +26,7 @@ public class Module : XmppStreamModule { public override void detach(XmppStream stream) { stream.stream_negotiated.disconnect(enable); stream.get_module(MessageModule.IDENTITY).received_pipeline.disconnect(received_pipeline_listener); + stream.get_module(ServiceDiscovery.Module.IDENTITY).remove_feature(stream, NS_URI); } public override string get_ns() { diff --git a/xmpp-vala/src/module/xep/0308_last_message_correction.vala b/xmpp-vala/src/module/xep/0308_last_message_correction.vala index 7669b244..41237660 100644 --- a/xmpp-vala/src/module/xep/0308_last_message_correction.vala +++ b/xmpp-vala/src/module/xep/0308_last_message_correction.vala @@ -21,7 +21,9 @@ public class Module : XmppStreamModule { stream.get_module(ServiceDiscovery.Module.IDENTITY).add_feature(stream, NS_URI); } - public override void detach(XmppStream stream) {} + public override void detach(XmppStream stream) { + stream.get_module(ServiceDiscovery.Module.IDENTITY).remove_feature(stream, NS_URI); + } public override string get_ns() { return NS_URI; } diff --git a/xmpp-vala/src/module/xep/0333_chat_markers.vala b/xmpp-vala/src/module/xep/0333_chat_markers.vala index 7f92fbe3..44b2ddb7 100644 --- a/xmpp-vala/src/module/xep/0333_chat_markers.vala +++ b/xmpp-vala/src/module/xep/0333_chat_markers.vala @@ -36,8 +36,9 @@ public class Module : XmppStreamModule { } public override void detach(XmppStream stream) { - stream.get_module(MessageModule.IDENTITY).received_message.disconnect(on_received_message); + stream.get_module(ServiceDiscovery.Module.IDENTITY).remove_feature(stream, NS_URI); stream.get_module(MessageModule.IDENTITY).send_pipeline.disconnect(send_pipeline_listener); + stream.get_module(MessageModule.IDENTITY).received_message.disconnect(on_received_message); } public override string get_ns() { return NS_URI; } diff --git a/xmpp-vala/src/module/xep/0359_unique_stable_stanza_ids.vala b/xmpp-vala/src/module/xep/0359_unique_stable_stanza_ids.vala index b6defc6d..861e9bcc 100644 --- a/xmpp-vala/src/module/xep/0359_unique_stable_stanza_ids.vala +++ b/xmpp-vala/src/module/xep/0359_unique_stable_stanza_ids.vala @@ -14,7 +14,9 @@ public class Module : XmppStreamModule { stream.get_module(ServiceDiscovery.Module.IDENTITY).add_feature(stream, NS_URI); } - public override void detach(XmppStream stream) {} + public override void detach(XmppStream stream) { + stream.get_module(ServiceDiscovery.Module.IDENTITY).remove_feature(stream, NS_URI); + } public override string get_ns() { return NS_URI; } diff --git a/xmpp-vala/src/module/xep/0391_jingle_encrypted_transports.vala b/xmpp-vala/src/module/xep/0391_jingle_encrypted_transports.vala index a6827bf3..1161ec96 100644 --- a/xmpp-vala/src/module/xep/0391_jingle_encrypted_transports.vala +++ b/xmpp-vala/src/module/xep/0391_jingle_encrypted_transports.vala @@ -15,6 +15,7 @@ public class Module : XmppStreamModule, SecurityPrecondition { } public override void detach(XmppStream stream) { + stream.get_module(ServiceDiscovery.Module.IDENTITY).remove_feature(stream, NS_URI); } public bool is_available(XmppStream stream, Jid full_jid) { @@ -139,4 +140,4 @@ public interface EnvelopEncoding : Object { public abstract void encode_envelop(XmppStream stream, Jid local_full_jid, Jid peer_full_jid, SecurityParameters security_params, StanzaNode security); } -} \ No newline at end of file +} diff --git a/xmpp-vala/src/module/xep/0402_bookmarks2.vala b/xmpp-vala/src/module/xep/0402_bookmarks2.vala index 563bc286..5641769a 100644 --- a/xmpp-vala/src/module/xep/0402_bookmarks2.vala +++ b/xmpp-vala/src/module/xep/0402_bookmarks2.vala @@ -101,7 +101,9 @@ public class Module : BookmarksProvider, XmppStreamModule { stream.get_module(Pubsub.Module.IDENTITY).add_filtered_notification(stream, NS_URI, on_pupsub_item, on_pupsub_retract); } - public override void detach(XmppStream stream) { } + public override void detach(XmppStream stream) { + stream.get_module(Pubsub.Module.IDENTITY).remove_filtered_notification(stream, NS_URI); + } public override string get_ns() { return NS_URI; } public override string get_id() { return IDENTITY.id; } -- cgit v1.2.3-70-g09d2