Commits

commits@gajim.org

17632 discussions

[Git][gajim/gajim][master] 2 commits: Use Gio.NetworkMonitor for reconnecting
by Philipp Hörist 18 Nov '17

18 Nov '17

Philipp Hörist pushed to branch master at gajim / gajim Commits: e8004084 by Philipp Hörist at 2017-11-12T23:53:03+01:00 Use Gio.NetworkMonitor for reconnecting - This uses also NetworkManager, but also other more low level interfaces on linux - This should work cross-platform - Leave network_watcher.py for now, because it shows how to use the Gtk dbus interface - - - - - f7bb9c8a by Philipp Hörist at 2017-11-18T22:09:35+01:00 Merge branch 'networkmonitor' into 'master' Use Gio.NetworkMonitor for reconnecting See merge request gajim/gajim!149 - - - - - 2 changed files: - gajim/features_window.py - gajim/gui_interface.py Changes: ===================================== gajim/features_window.py ===================================== --- a/gajim/features_window.py +++ b/gajim/features_window.py @@ -57,10 +57,6 @@ class FeaturesWindow: _('Ability to encrypting chat messages with OpenPGP.'), _('Requires gpg and python-gnupg (http://code.google.com/p/python-gnupg/).'), _('Requires gpg.exe in PATH.')), - _('Network-Watcher'): (self.network_watcher_available, - _('Autodetection of network status.'), - _('Requires gnome-network-manager'), - _('Feature not available under Windows.')), _('Password encryption'): (self.some_keyring_available, _('Passwords can be stored securely and not just in plaintext.'), _('Requires libsecret and a provider (such as GNOME Keyring and KSecretService).'), @@ -157,10 +153,6 @@ class FeaturesWindow: def gpg_available(self): return app.HAVE_GPG - def network_watcher_available(self): - from gajim import network_watcher - return network_watcher.supported - def some_keyring_available(self): if os.name == 'nt': return True ===================================== gajim/gui_interface.py ===================================== --- a/gajim/gui_interface.py +++ b/gajim/gui_interface.py @@ -44,6 +44,7 @@ import hashlib from gi.repository import Gtk from gi.repository import GdkPixbuf from gi.repository import GLib +from gi.repository import Gio try: from PIL import Image @@ -2558,6 +2559,23 @@ class Interface: view.updateNamespace({'gajim': app}) app.ipython_window = window + def network_status_changed(self, monitor, connected): + if connected == self.network_state: + # This callback gets called a lot from GTK with the + # same state, not only on change. + return + self.network_state = connected + if connected: + for connection in app.connections.values(): + if connection.connected <= 0: + log.info('Connect %s', connection.name) + connection.reconnect() + else: + for connection in app.connections.values(): + if connection.connected > 1: + log.info('Disconnect %s', connection.name) + connection.disconnectedReconnCB() + def run(self, application): if app.config.get('trayicon') != 'never': self.show_systray() @@ -2779,8 +2797,6 @@ class Interface: self.remote_ctrl = None - from gajim import network_watcher - if dbus_support.supported: from gajim import upower_listener from gajim import logind_listener @@ -2882,6 +2898,11 @@ class Interface: self.music_track_changed_signal = None + self.network_monitor = Gio.NetworkMonitor.get_default() + self.network_monitor.connect('network-changed', + self.network_status_changed) + self.network_state = self.network_monitor.get_network_available() + class PassphraseRequest: def __init__(self, keyid): View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/639f0db1646ae5515f29996b8c7387b62… --- View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/639f0db1646ae5515f29996b8c7387b62… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] 2 commits: Add symbolic icon
by Yann Leboulanger 18 Nov '17

18 Nov '17

Yann Leboulanger pushed to branch master at gajim / gajim Commits: 566b5370 by André Apitzsch at 2017-11-17T23:04:05+01:00 Add symbolic icon - - - - - 639f0db1 by Yann Leboulanger at 2017-11-18T21:11:19+01:00 Merge branch 'symbolic' into 'master' Add symbolic icon See merge request gajim/gajim!151 - - - - - 2 changed files: - + gajim/data/icons/hicolor/symbolic/apps/org.gajim.Gajim-symbolic.svg - setup.py Changes: ===================================== gajim/data/icons/hicolor/symbolic/apps/org.gajim.Gajim-symbolic.svg ===================================== --- /dev/null +++ b/gajim/data/icons/hicolor/symbolic/apps/org.gajim.Gajim-symbolic.svg @@ -0,0 +1,129 @@ +<?xml version="1.0" encoding="UTF-8" standalone="no"?> + + +<svg + xmlns:dc="http://purl.org/dc/elements/1.1/" + xmlns:cc="http://creativecommons.org/ns#" + xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#" + xmlns:svg="http://www.w3.org/2000/svg" + xmlns="http://www.w3.org/2000/svg" + xmlns:sodipodi="http://sodipodi.sourceforge.net/DTD/sodipodi-0.dtd" + xmlns:inkscape="http://www.inkscape.org/namespaces/inkscape" + width="16" + height="16" + id="svg4023" + sodipodi:version="0.32" + inkscape:version="0.92.2 5c3e80d, 2017-08-06" + sodipodi:docname="org.gajim.Gajim-symbolic.svg" + inkscape:output_extension="org.inkscape.output.svg.inkscape" + inkscape:export-filename="/home/cornelius/gajim.png" + inkscape:export-xdpi="90" + inkscape:export-ydpi="90" + version="1.1" + viewBox="-0.7 0.2 16 16"> + <defs + id="defs4025" /> + <sodipodi:namedview + id="base" + pagecolor="#555753" + bordercolor="#666666" + borderopacity="1.0" + inkscape:pageopacity="1" + inkscape:pageshadow="2" + inkscape:zoom="14" + inkscape:cx="-12.741964" + inkscape:cy="-2.4941427" + inkscape:current-layer="layer1" + showgrid="false" + inkscape:grid-bbox="true" + inkscape:document-units="px" + inkscape:window-width="1920" + inkscape:window-height="1016" + inkscape:window-x="0" + inkscape:window-y="27" + inkscape:window-maximized="1" + showguides="true" + fit-margin-top="0" + fit-margin-left="0" + fit-margin-right="0" + fit-margin-bottom="0" + showborder="true" + borderlayer="true" + viewbox-x="-0.7" + viewbox-y="0.2"> + <inkscape:grid + type="xygrid" + id="grid68" + originx="-4.9243182" + originy="-2.7517627" /> + </sodipodi:namedview> + <metadata + id="metadata4028"> + <rdf:RDF> + <cc:Work + rdf:about=""> + <dc:format>image/svg+xml</dc:format> + <dc:type + rdf:resource="http://purl.org/dc/dcmitype/StillImage" /> + <dc:title /> + <dc:date /> + <dc:creator> + <cc:Agent> + <dc:title>Josef Vybíral</dc:title> + </cc:Agent> + </dc:creator> + <dc:rights> + <cc:Agent> + <dc:title /> + </cc:Agent> + </dc:rights> + <dc:publisher> + <cc:Agent> + <dc:title /> + </cc:Agent> + </dc:publisher> + <dc:source /> + <dc:subject> + <rdf:Bag> + <rdf:li>gajim</rdf:li> + <rdf:li>jabber</rdf:li> + <rdf:li>im</rdf:li> + </rdf:Bag> + </dc:subject> + <cc:license + rdf:resource="http://creativecommons.org/licenses/by/2.5/" /> + </cc:Work> + <cc:License + rdf:about="http://creativecommons.org/licenses/by/2.5/"> + <cc:permits + rdf:resource="http://web.resource.org/cc/Reproduction" /> + <cc:permits + rdf:resource="http://web.resource.org/cc/Distribution" /> + <cc:requires + rdf:resource="http://web.resource.org/cc/Notice" /> + <cc:requires + rdf:resource="http://web.resource.org/cc/Attribution" /> + <cc:permits + rdf:resource="http://web.resource.org/cc/DerivativeWorks" /> + </cc:License> + </rdf:RDF> + </metadata> + <g + id="layer1" + inkscape:label="Layer 1" + inkscape:groupmode="layer" + transform="translate(-4.9243182,-29.248236)"> + <path + sodipodi:nodetypes="csccccccsc" + id="path2200" + d="m 12.431363,35.902348 c -3.596104,0 -7.047591,2.144734 -6.802621,2.810805 0.03381,0.09192 2.338952,0.109837 2.419739,0.623857 l 0.665998,4.02844 c 0,0.384169 0.02928,0.797428 0.285041,0.797428 2.391902,0.539175 4.672155,0.520362 6.852052,0 0.255764,0 0.296674,-0.300871 0.296675,-0.68504 l 0.637887,-4.112731 c 0.107379,-0.43303 2.336693,-0.339017 2.39163,-0.764343 0.103066,-0.797944 -3.150297,-2.698416 -6.746401,-2.698416 z" + style="color:#000000;display:inline;overflow:visible;visibility:visible;opacity:1;fill:#000000;fill-opacity:0;fill-rule:nonzero;stroke:#bebebe;stroke-width:1.5;stroke-linecap:butt;stroke-linejoin:miter;stroke-miterlimit:4;stroke-dasharray:none;stroke-dashoffset:0;stroke-opacity:1;marker:none;marker-start:none;marker-mid:none;marker-end:none;image-rendering:auto;enable-background:accumulate" + inkscape:connector-curvature="0" /> + <path + sodipodi:nodetypes="csccccccsc" + id="path3941" + d="m 12.431363,30.392789 c -1.630186,0 -2.951551,1.371963 -2.951551,3.062415 0,1.080401 0.589091,1.867472 1.501919,2.57612 0.532203,0.629506 0.3362,0.543594 0.376381,1.202316 0,0.204542 0.06765,0.267362 0.26479,0.267362 0.55232,0.501159 1.118692,0.488201 1.649449,-0.01634 0.197139,0 0.215511,-0.09158 0.215511,-0.296119 0.06876,-0.853384 0.107094,-0.75079 0.487008,-1.255105 0.842182,-0.53965 1.408045,-1.376645 1.408045,-2.478234 0,-1.690453 -1.321366,-3.062416 -2.951552,-3.062415 z" + style="color:#000000;display:inline;overflow:visible;visibility:visible;opacity:1;fill:#000000;fill-opacity:0;fill-rule:nonzero;stroke:#bebebe;stroke-width:1.5;stroke-linecap:square;stroke-linejoin:miter;stroke-miterlimit:4;stroke-dasharray:none;stroke-dashoffset:0;stroke-opacity:1;marker:none;marker-start:none;marker-mid:none;marker-end:none;enable-background:accumulate" + inkscape:connector-curvature="0" /> + </g> +</svg> ===================================== setup.py ===================================== --- a/setup.py +++ b/setup.py @@ -238,7 +238,9 @@ data_files_app_icon = [ ("share/icons/hicolor/128x128/apps", ["gajim/data/icons/hicolor/128x128/apps/org.gajim.Gajim.png"]), ("share/icons/hicolor/scalable/apps", - ["gajim/data/icons/hicolor/scalable/apps/org.gajim.Gajim.svg"]) + ["gajim/data/icons/hicolor/scalable/apps/org.gajim.Gajim.svg"]), + ("share/icons/hicolor/symbolic/apps", + ["gajim/data/icons/hicolor/symbolic/apps/org.gajim.Gajim-symbolic.svg"]) ] data_files = data_files_app_icon View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/54f4c36fe3ade18b267eb7169982d2911… --- View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/54f4c36fe3ade18b267eb7169982d2911… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] 2 commits: MUC: Improve error messages
by Philipp Hörist 18 Nov '17

18 Nov '17

Philipp Hörist pushed to branch master at gajim / gajim Commits: 86f5953e by Annika Sommer at 2017-11-18T18:10:10+01:00 MUC: Improve error messages Distinguish between 'item-not-found' and 'remote-server-not-found' Fixes #8749 - - - - - 54f4c36f by Philipp Hörist at 2017-11-18T18:18:41+01:00 Merge branch 'master' into 'master' Fix #8749: Misleading error message when joining a room Closes #8749 See merge request gajim/gajim!150 - - - - - 1 changed file: - gajim/gui_interface.py Changes: ===================================== gajim/gui_interface.py ===================================== --- a/gajim/gui_interface.py +++ b/gajim/gui_interface.py @@ -326,8 +326,13 @@ class Interface: obj.room_jid) elif (obj.errcode == '404') or (obj.errcon in ('item-not-found', 'remote-server-not-found')): + # remote server does not exist + if (obj.errcon == 'remote-server-not-found'): + self.handle_gc_error(gc_control, _('Unable to join group chat'), + _('Remote server <b>%s</b> does not exist.') % obj.room_jid) # group chat does not exist - self.handle_gc_error(gc_control, _('Unable to join group chat'), + else: + self.handle_gc_error(gc_control, _('Unable to join group chat'), _('Group chat <b>%s</b> does not exist.') % obj.room_jid) elif (obj.errcode == '405') or (obj.errcon == 'not-allowed'): self.handle_gc_error(gc_control, _('Unable to join group chat'), View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/c1c2a92efccbc89e625e27ff5129bb498… --- View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/c1c2a92efccbc89e625e27ff5129bb498… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Init Plugins and Caps after Roster
by Philipp Hörist 18 Nov '17

18 Nov '17

Philipp Hörist pushed to branch master at gajim / gajim Commits: c1c2a92e by Philipp Hörist at 2017-11-18T11:11:47+01:00 Init Plugins and Caps after Roster - Some plugins do not depend on gui extension points and need the roster to be initialized - - - - - 1 changed file: - gajim/gui_interface.py Changes: ===================================== gajim/gui_interface.py ===================================== --- a/gajim/gui_interface.py +++ b/gajim/gui_interface.py @@ -2562,6 +2562,20 @@ class Interface: MessageWindowMgr.ONE_MSG_WINDOW_ALWAYS_WITH_ROSTER: self.msg_win_mgr.create_window(None, None, None) + # Creating plugin manager + from gajim import plugins + app.plugin_manager = plugins.PluginManager() + app.plugin_manager.init_plugins() + + helpers.update_optional_features() + # prepopulate data which we are sure of; note: we do not log these info + for account in app.connections: + gajimcaps = caps_cache.capscache[ + ('sha-1', app.caps_hash[account])] + gajimcaps.identities = [app.gajim_identity] + gajimcaps.features = app.gajim_common_features + \ + app.gajim_optional_features[account] + self.roster._before_fill() for account in app.connections: app.connections[account].load_roster_from_db() @@ -2758,20 +2772,6 @@ class Interface: app.gajim_optional_features[a] = [] app.caps_hash[a] = '' - # Creating plugin manager - from gajim import plugins - app.plugin_manager = plugins.PluginManager() - app.plugin_manager.init_plugins() - - helpers.update_optional_features() - # prepopulate data which we are sure of; note: we do not log these info - for account in app.connections: - gajimcaps = caps_cache.capscache[('sha-1', - app.caps_hash[account])] - gajimcaps.identities = [app.gajim_identity] - gajimcaps.features = app.gajim_common_features + \ - app.gajim_optional_features[account] - self.remote_ctrl = None from gajim import network_watcher View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/c1c2a92efccbc89e625e27ff5129bb4984… --- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/c1c2a92efccbc89e625e27ff5129bb4984… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Support unpublishing avatars (with XEP-0084)
by Philipp Hörist 17 Nov '17

17 Nov '17

Philipp Hörist pushed to branch master at gajim / gajim Commits: a80f41e2 by Philipp Hörist at 2017-11-17T19:19:59+01:00 Support unpublishing avatars (with XEP-0084) - Add more logging - - - - - 2 changed files: - gajim/common/connection_handlers.py - gajim/common/pep.py Changes: ===================================== gajim/common/connection_handlers.py ===================================== --- a/gajim/common/connection_handlers.py +++ b/gajim/common/connection_handlers.py @@ -277,13 +277,17 @@ class ConnectionVcard: if self.get_own_jid().bareMatch(obj.jid): app.log('avatar').info('Update (vCard): %s %s', - obj.jid, obj.avatar_sha) + obj.jid, obj.avatar_sha) current_sha = app.config.get_per( 'accounts', self.name, 'avatar_sha') if obj.avatar_sha != current_sha: app.log('avatar').info( 'Request (vCard): %s', obj.jid) self.request_vcard(self._on_own_avatar_received) + else: + app.log('avatar').info( + 'Avatar already known (vCard): %s %s', + obj.jid, obj.avatar_sha) return if obj.avatar_sha == '': @@ -305,6 +309,10 @@ class ConnectionVcard: app.log('avatar').info( 'Request (vCard): %s', obj.jid) self.request_vcard(self._on_avatar_received, obj.jid) + else: + app.log('avatar').info( + 'Avatar already known (vCard): %s %s', + obj.jid, obj.avatar_sha) def _vcard_gc_presence_received(self, obj): if obj.conn.name != self.name: @@ -348,6 +356,9 @@ class ConnectionVcard: obj.nick, obj.avatar_sha) gc_contact.avatar_sha = obj.avatar_sha app.interface.update_avatar(contact=gc_contact) + else: + app.log('avatar').info( + 'Avatar already known (vCard): %s', obj.nick) def send_avatar_presence(self): show = helpers.get_xmpp_show(app.SHOW_LIST[self.connected]) ===================================== gajim/common/pep.py ===================================== --- a/gajim/common/pep.py +++ b/gajim/common/pep.py @@ -481,23 +481,36 @@ class AvatarNotificationPEP(AbstractPEP): namespace = 'urn:xmpp:avatar:metadata' def _extract_info(self, items): - avatar = None + self.avatar = None for item in items.getTags('item'): info = item.getTag('metadata').getTag('info') - self.avatar = info.getAttrs() + if info is not None: + self.avatar = info.getAttrs() break - return (avatar, False) + return (None, False) def _on_receive(self, jid, account): - sha = app.contacts.get_avatar_sha(account, jid) - app.log('avatar').info( - 'Update (Pubsub): %s %s', jid, self.avatar['id']) - if sha == self.avatar['id']: - return con = app.connections[account] - app.log('avatar').info('Request (Pubsub): %s', jid) - con.send_pb_retrieve(jid, 'urn:xmpp:avatar:data', self.avatar['id']) + if self.avatar is None: + # Remove avatar + app.log('avatar').debug('Remove (Pubsub): %s', jid) + app.contacts.set_avatar(account, jid, None) + own_jid = con.get_own_jid().getStripped() + app.logger.set_avatar_sha(own_jid, jid, None) + app.interface.update_avatar(account, jid) + else: + sha = app.contacts.get_avatar_sha(account, jid) + app.log('avatar').info( + 'Update (Pubsub): %s %s', jid, self.avatar['id']) + if sha == self.avatar['id']: + app.log('avatar').info( + 'Avatar already known (Pubsub): %s %s', + jid, self.avatar['id']) + return + app.log('avatar').info('Request (Pubsub): %s', jid) + con.send_pb_retrieve(jid, 'urn:xmpp:avatar:data', + self.avatar['id']) SUPPORTED_PERSONAL_USER_EVENTS = [ View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/a80f41e2bf431a5b4c2c32563d0775b76d… --- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/a80f41e2bf431a5b4c2c32563d0775b76d… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] 4 commits: Init PluginManger earlier
by Philipp Hörist 15 Nov '17

15 Nov '17

Philipp Hörist pushed to branch master at gajim / gajim Commits: 0fe78daa by Philipp Hörist at 2017-11-15T21:57:15+01:00 Init PluginManger earlier - Plugins need the possibility to add caps before Gajim updates them - - - - - 775fec2f by Philipp Hörist at 2017-11-15T22:26:55+01:00 Dont activate plugins in PluginManager init Plugins maybe want to use in their activation Gajim methods that use an extension point. But because the PluginManager Object is not fully initialized, the extension point method is not available yet First create the PluginManager Object, then load Plugins. - - - - - d5ddbe37 by Philipp Hörist at 2017-11-15T22:28:11+01:00 Add extension point for plugins to add their caps - - - - - 9f0154ef by Philipp Hörist at 2017-11-15T22:28:57+01:00 Add caps to first presence According to the XEP, caps hash has to be added to all presences - - - - - 4 changed files: - gajim/common/connection_handlers.py - gajim/common/helpers.py - gajim/gui_interface.py - gajim/plugins/pluginmanager.py Changes: ===================================== gajim/common/connection_handlers.py ===================================== --- a/gajim/common/connection_handlers.py +++ b/gajim/common/connection_handlers.py @@ -2039,6 +2039,7 @@ ConnectionHandlersBase, ConnectionJingle, ConnectionIBBytestream): p.setStatus(msg) if signed: p.setTag(nbxmpp.NS_SIGNED + ' x').setData(signed) + p = self.add_sha(p) if self.connection: self.connection.send(p) ===================================== gajim/common/helpers.py ===================================== --- a/gajim/common/helpers.py +++ b/gajim/common/helpers.py @@ -1354,6 +1354,10 @@ def update_optional_features(account = None): app.gajim_optional_features[a].append(nbxmpp.NS_JINGLE_XTLS) app.gajim_optional_features[a].append(nbxmpp.NS_JINGLE_BYTESTREAM) app.gajim_optional_features[a].append(nbxmpp.NS_JINGLE_IBB) + + # Give plugins the possibility to add their features + app.plugin_manager.extension_point('update_caps', a) + app.caps_hash[a] = caps_cache.compute_caps_hash([app.gajim_identity], app.gajim_common_features + app.gajim_optional_features[a]) # re-send presence with new hash ===================================== gajim/gui_interface.py ===================================== --- a/gajim/gui_interface.py +++ b/gajim/gui_interface.py @@ -2562,10 +2562,6 @@ class Interface: MessageWindowMgr.ONE_MSG_WINDOW_ALWAYS_WITH_ROSTER: self.msg_win_mgr.create_window(None, None, None) - # Creating plugin manager - from gajim import plugins - app.plugin_manager = plugins.PluginManager() - self.roster._before_fill() for account in app.connections: app.connections[account].load_roster_from_db() @@ -2762,6 +2758,11 @@ class Interface: app.gajim_optional_features[a] = [] app.caps_hash[a] = '' + # Creating plugin manager + from gajim import plugins + app.plugin_manager = plugins.PluginManager() + app.plugin_manager.init_plugins() + helpers.update_optional_features() # prepopulate data which we are sure of; note: we do not log these info for account in app.connections: ===================================== gajim/plugins/pluginmanager.py ===================================== --- a/gajim/plugins/pluginmanager.py +++ b/gajim/plugins/pluginmanager.py @@ -110,7 +110,6 @@ class PluginManager(metaclass=Singleton): for path in [app.PLUGINS_DIRS[1], app.PLUGINS_DIRS[0]]: pc = PluginManager.scan_dir_for_plugins(path) self.add_plugins(pc) - self._activate_all_plugins_from_global_config() @log_calls('PluginManager') def _plugin_has_entry_in_global_config(self, plugin): @@ -127,6 +126,10 @@ class PluginManager(metaclass=Singleton): app.config.del_per('plugins', plugin.short_name) @log_calls('PluginManager') + def init_plugins(self): + self._activate_all_plugins_from_global_config() + + @log_calls('PluginManager') def add_plugin(self, plugin_class): ''' :todo: what about adding plug-ins that are already added? Module reload View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/5236693df9f5855888411bcaaed41b01f… --- View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/5236693df9f5855888411bcaaed41b01f… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/python-nbxmpp][master] fix tests
by Yann Leboulanger 14 Nov '17

14 Nov '17

Yann Leboulanger pushed to branch master at gajim / python-nbxmpp Commits: 6807867b by Yann Leboulanger at 2017-11-14T22:17:29+01:00 fix tests - - - - - 1 changed file: - test/unit/test_xmpp_transports_nb2.py Changes: ===================================== test/unit/test_xmpp_transports_nb2.py ===================================== --- a/test/unit/test_xmpp_transports_nb2.py +++ b/test/unit/test_xmpp_transports_nb2.py @@ -231,9 +231,9 @@ class TestNonBlockingHTTP(AbstractTransportTest): transport = self._get_transport(self.bosh_http_dict) data = "<test>Please don't fail!</test>" - http_message = transport.build_http_message(data) - statusline, headers, http_body, buffer_rest = transport.parse_http_message( - http_message) + http_message = transport.build_http_message(data.encode('utf-8')) + statusline, headers, http_body, buffer_rest = \ + transport.parse_http_message(http_message.decode('utf-8')) self.assertFalse(bool(buffer_rest)) self.assertTrue(statusline and isinstance(statusline, list)) View it on GitLab: https://dev.gajim.org/gajim/python-nbxmpp/commit/6807867b231ed7e25fbdbac364… --- View it on GitLab: https://dev.gajim.org/gajim/python-nbxmpp/commit/6807867b231ed7e25fbdbac364… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/python-nbxmpp][master] fix BOSH connexion
by Yann Leboulanger 14 Nov '17

14 Nov '17

Yann Leboulanger pushed to branch master at gajim / python-nbxmpp Commits: bda75fdf by Yann Leboulanger at 2017-11-14T17:57:40+01:00 fix BOSH connexion - - - - - 2 changed files: - nbxmpp/bosh.py - nbxmpp/transports_nb.py Changes: ===================================== nbxmpp/bosh.py ===================================== --- a/nbxmpp/bosh.py +++ b/nbxmpp/bosh.py @@ -547,10 +547,10 @@ class KeyStack(object): self.first_call = True def reset(self): - seed = str(get_rand_number()) + seed = str(get_rand_number()).encode('utf-8') self.keys = [sha1(seed).hexdigest()] for i in range(self.count-1): - curr_seed = self.keys[i] + curr_seed = self.keys[i].encode('utf-8') self.keys.append(sha1(curr_seed).hexdigest()) def get(self): ===================================== nbxmpp/transports_nb.py ===================================== --- a/nbxmpp/transports_nb.py +++ b/nbxmpp/transports_nb.py @@ -784,7 +784,7 @@ class NonBlockingHTTP(NonBlockingTCP): 'Host: %s:%s' % (self.http_host, self.http_port), 'User-Agent: Gajim', 'Content-Type: text/xml; charset=utf-8', - 'Content-Length: %s' % len(str(httpbody))] + 'Content-Length: %s' % len(httpbody)] if self.add_proxy_headers: headers.append('Proxy-Connection: keep-alive') headers.append('Pragma: no-cache') @@ -796,7 +796,7 @@ class NonBlockingHTTP(NonBlockingTCP): headers.append('Connection: Keep-Alive') headers.append('\r\n') headers = '\r\n'.join(headers) - return '%s%s' % (headers, httpbody) + return b'%s%s' % (headers.encode('utf-8'), httpbody) def parse_http_message(self, message): """ View it on GitLab: https://dev.gajim.org/gajim/python-nbxmpp/commit/bda75fdfc0d0fd37370cd63afa… --- View it on GitLab: https://dev.gajim.org/gajim/python-nbxmpp/commit/bda75fdfc0d0fd37370cd63afa… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Parse stanza-id in groupchat correctly
by Philipp Hörist 11 Nov '17

11 Nov '17

Philipp Hörist pushed to branch master at gajim / gajim Commits: 5236693d by Philipp Hörist at 2017-11-11T21:46:00+01:00 Parse stanza-id in groupchat correctly - - - - - 1 changed file: - gajim/common/connection_handlers_events.py Changes: ===================================== gajim/common/connection_handlers_events.py ===================================== --- a/gajim/common/connection_handlers_events.py +++ b/gajim/common/connection_handlers_events.py @@ -148,10 +148,14 @@ class HelperEvent: if by is None: # We can not verify who set this stanza-id, ignore it. return - elif not self.conn.get_own_jid().bareMatch(by): - # by attribute does not match the server, ignore it. - return - return stanza_id + if stanza.getType() == 'groupchat': + if stanza.getFrom().bareMatch(by): + # by attribute must match the server + return stanza_id + elif self.conn.get_own_jid().bareMatch(by): + # by attribute must match the server + return stanza_id + return @staticmethod def get_forwarded_message(stanza): View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/5236693df9f5855888411bcaaed41b01f3… --- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/5236693df9f5855888411bcaaed41b01f3… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Escape account label in roster and menus
by Philipp Hörist 11 Nov '17

11 Nov '17

Philipp Hörist pushed to branch master at gajim / gajim Commits: 309e844b by Philipp Hörist at 2017-11-11T17:10:31+01:00 Escape account label in roster and menus - - - - - 2 changed files: - gajim/gui_menu_builder.py - gajim/roster_window.py Changes: ===================================== gajim/gui_menu_builder.py ===================================== --- a/gajim/gui_menu_builder.py +++ b/gajim/gui_menu_builder.py @@ -780,7 +780,8 @@ def build_accounts_menu(): return if len(accounts_list) > 1: for acc in accounts_list: - label = app.config.get_per('accounts', acc, 'account_label') + label = escape_mnemonic( + app.config.get_per('accounts', acc, 'account_label')) acc_menu.append_submenu( label or acc, get_account_menu(acc)) else: @@ -806,8 +807,8 @@ def build_bookmark_menu(account): if acc_menu.get_item_link(0, 'submenu'): for i in range(acc_menu.get_n_items()): label = acc_menu.get_item_attribute_value(i, 'label') - account_label = app.config.get_per('accounts', account, - 'account_label') + account_label = escape_mnemonic( + app.config.get_per('accounts', account, 'account_label')) if label.get_string() in (account_label, account): menu = acc_menu.get_item_link(i, 'submenu') else: @@ -835,3 +836,11 @@ def get_encryption_menu(control_id, type_id): if menu.get_n_items() == 1: return None return menu + + +def escape_mnemonic(label): + if label is None: + return + # Underscore inside a label means the next letter is a keyboard + # shortcut. To show an underscore we have to use double underscore + return label.replace('_', '__') ===================================== gajim/roster_window.py ===================================== --- a/gajim/roster_window.py +++ b/gajim/roster_window.py @@ -1069,7 +1069,7 @@ class RosterWindow: accounts = accounts) account_name += ' (%s/%s)' % (repr(nbr_on), repr(nbr_total)) - self.model[child_iter][Column.NAME] = account_name + self.model[child_iter][Column.NAME] = GLib.markup_escape_text(account_name) pep_dict = app.connections[account].pep if app.config.get('show_mood_in_roster') and 'mood' in pep_dict: View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/309e844b4ee853c1dbb17108e517ff38e9… --- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/309e844b4ee853c1dbb17108e517ff38e9… You're receiving this email because of your account on dev.gajim.org.

1 0

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

Commits