Commits September 2019

commits@gajim.org

1 participants
59 discussions

[Git][gajim/gajim][master] Jingle: Fix undefined variable
by Philipp Hörist 29 Sep '19

29 Sep '19

Philipp Hörist pushed to branch master at gajim / gajim Commits: f70365da by Philipp Hörist at 2019-09-29T16:19:47Z Jingle: Fix undefined variable - - - - - 1 changed file: - gajim/common/modules/jingle.py Changes: ===================================== gajim/common/modules/jingle.py ===================================== @@ -245,7 +245,7 @@ class Jingle(BaseModule): if media: if media not in ('audio', 'video', 'file'): return [] - return [se for se in sessions if session.get_content(media)] + return [se for se in sessions if se.get_content(media)] return sessions def set_file_info(self, file_): View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/f70365da8e7f57c3aa1af6d70f98793dd0… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/f70365da8e7f57c3aa1af6d70f98793dd0… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Bookmarks: Add dev config setting for testing bookmarks 2
by Philipp Hörist 29 Sep '19

29 Sep '19

Philipp Hörist pushed to branch master at gajim / gajim Commits: a934551c by Philipp Hörist at 2019-09-29T11:32:03Z Bookmarks: Add dev config setting for testing bookmarks 2 - - - - - 2 changed files: - gajim/common/config.py - gajim/common/modules/bookmarks.py Changes: ===================================== gajim/common/config.py ===================================== @@ -289,6 +289,7 @@ class Config: 'auto_copy': [opt_bool, True, _('Selecting text will copy it to the clipboard')], 'command_system_execute': [opt_bool, False, _('If enabled, Gajim will execute commands (/show, /sh, /execute, /exec).')], 'groupchat_roster_width': [opt_int, 210, _('Width of group chat roster in pixel')], + 'dev_force_bookmark_2': [opt_bool, False, _('Force Bookmark 2 usage')], }, {}) # type: Tuple[Dict[str, List[Any]], Dict[Any, Any]] __options_per_key = { ===================================== gajim/common/modules/bookmarks.py ===================================== @@ -155,7 +155,12 @@ class Bookmarks(BaseModule): NetworkEvent('bookmarks-received', account=self._account)) def pass_disco(self, info): - if nbxmpp.NS_BOOKMARKS_COMPAT in info.features: + if app.config.get('dev_force_bookmark_2'): + self._log.info('Forcing Bookmark 2 usage, ' + 'without server conversion support: %s', info.jid) + self._conversion_2 = True + + elif nbxmpp.NS_BOOKMARKS_COMPAT in info.features: self._conversion_2 = True self._log.info('Discovered Bookmarks Conversion 2: %s', info.jid) View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/a934551cb1b643724f25b83db073c2cc38… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/a934551cb1b643724f25b83db073c2cc38… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] 2 commits: Remove unused function
by Philipp Hörist 29 Sep '19

29 Sep '19

Philipp Hörist pushed to branch master at gajim / gajim Commits: 1f663a24 by Philipp Hörist at 2019-09-29T08:35:03Z Remove unused function - - - - - f5244e37 by Philipp Hörist at 2019-09-29T11:20:16Z Bookmarks: Don't accept PEP notificiations before request - - - - - 2 changed files: - gajim/common/modules/adhoc_commands.py - gajim/common/modules/bookmarks.py Changes: ===================================== gajim/common/modules/adhoc_commands.py ===================================== @@ -179,24 +179,6 @@ class ChangeStatusCommand(AdHocCommand): return False # finish the session -def find_current_groupchats(account): - from gajim import message_control - rooms = [] - for gc_control in app.interface.msg_win_mgr.get_controls( - message_control.TYPE_GC) + \ - app.interface.minimized_controls[account].values(): - acct = gc_control.account - # check if account is the good one - if acct != account: - continue - room_jid = gc_control.room_jid - nick = gc_control.nick - if (room_jid in app.gc_connected[acct] and - app.gc_connected[acct][room_jid]): - rooms.append((room_jid, nick,)) - return rooms - - class AdHocCommands(BaseModule): _nbxmpp_extends = 'AdHoc' ===================================== gajim/common/modules/bookmarks.py ===================================== @@ -48,7 +48,7 @@ class Bookmarks(BaseModule): self._conversion_2 = False self._bookmarks = [] self._join_timeouts = [] - self._request_in_progress = False + self._request_in_progress = True @property def conversion(self): View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/8936a07e46a0e54bb4a43dcb6cd907286… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/8936a07e46a0e54bb4a43dcb6cd907286… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] ServerInfo: Rework Dialog
by Philipp Hörist 29 Sep '19

29 Sep '19

Philipp Hörist pushed to branch master at gajim / gajim Commits: 8936a07e by Daniel Brötzmann at 2019-09-29T00:31:30Z ServerInfo: Rework Dialog - Split information into different notebook tabs - Show connection information (Fixes #6096) - - - - - 9 changed files: - gajim/app_actions.py - gajim/common/connection.py - gajim/data/gui/account_context_menu.ui - gajim/data/gui/certificate_dialog.ui - + gajim/data/gui/server_info.ui - gajim/data/style/gajim.css - gajim/gtk/dialogs.py - gajim/gtk/server_info.py - gajim/roster_window.py Changes: ===================================== gajim/app_actions.py ===================================== @@ -27,7 +27,7 @@ import gajim.plugins.gui from gajim.gtk.dialogs import ShortcutsWindow from gajim.gtk.history_sync import HistorySyncAssistant -from gajim.gtk.server_info import ServerInfoDialog +from gajim.gtk.server_info import ServerInfo from gajim.gtk.mam_preferences import MamPreferences from gajim.gtk.preferences import Preferences from gajim.gtk.groupchat_creation import CreateGroupchatWindow @@ -241,11 +241,11 @@ def on_privacy_lists(action, param): def on_server_info(action, param): account = param.get_string() - if 'server_info' in interface.instances[account]: - interface.instances[account]['server_info'].present() + window = app.get_app_window(ServerInfo, account) + if window is None: + ServerInfo(account) else: - interface.instances[account]['server_info'] = \ - ServerInfoDialog(account) + window.present() def on_xml_console(action, param): ===================================== gajim/common/connection.py ===================================== @@ -1018,6 +1018,9 @@ class Connection(CommonConnection, ConnectionHandlers): msg = '%s over proxy %s:%s' % (msg, self._proxy['host'], self._proxy['port']) log.info(msg) + def get_connection_info(self): + return self._current_host, self._proxy + def _connect_failure(self, con_type=None): if not con_type: # we are not retrying, and not conecting ===================================== gajim/data/gui/account_context_menu.ui ===================================== @@ -59,10 +59,10 @@ </object> </child> <child> - <object class="GtkMenuItem" id="view_certificate_menuitem"> + <object class="GtkMenuItem" id="view_server_info_menuitem"> <property name="visible">True</property> <property name="can_focus">False</property> - <property name="label" translatable="yes">_View Certificate</property> + <property name="label" translatable="yes">_View Server Info</property> <property name="use_underline">True</property> </object> </child> ===================================== gajim/data/gui/certificate_dialog.ui ===================================== @@ -79,6 +79,7 @@ <property name="can_focus">False</property> <property name="halign">start</property> <property name="label"><></property> + <property name="xalign">0</property> </object> <packing> <property name="left_attach">1</property> @@ -106,6 +107,7 @@ <property name="can_focus">False</property> <property name="halign">start</property> <property name="label"><></property> + <property name="xalign">0</property> </object> <packing> <property name="left_attach">1</property> @@ -133,6 +135,7 @@ <property name="can_focus">False</property> <property name="halign">start</property> <property name="label"><></property> + <property name="xalign">0</property> </object> <packing> <property name="left_attach">1</property> @@ -163,7 +166,8 @@ <property name="wrap">True</property> <property name="wrap_mode">char</property> <property name="selectable">True</property> - <property name="max_width_chars">21</property> + <property name="max_width_chars">30</property> + <property name="xalign">0</property> </object> <packing> <property name="left_attach">1</property> @@ -191,6 +195,7 @@ <property name="can_focus">False</property> <property name="halign">start</property> <property name="label"><></property> + <property name="xalign">0</property> </object> <packing> <property name="left_attach">1</property> @@ -218,6 +223,7 @@ <property name="can_focus">False</property> <property name="halign">start</property> <property name="label"><></property> + <property name="xalign">0</property> </object> <packing> <property name="left_attach">1</property> @@ -245,6 +251,7 @@ <property name="can_focus">False</property> <property name="halign">start</property> <property name="label"><></property> + <property name="xalign">0</property> </object> <packing> <property name="left_attach">1</property> @@ -272,6 +279,7 @@ <property name="can_focus">False</property> <property name="halign">start</property> <property name="label"><></property> + <property name="xalign">0</property> </object> <packing> <property name="left_attach">1</property> @@ -299,6 +307,7 @@ <property name="can_focus">False</property> <property name="halign">start</property> <property name="label"><></property> + <property name="xalign">0</property> </object> <packing> <property name="left_attach">1</property> @@ -347,6 +356,7 @@ <property name="wrap">True</property> <property name="wrap_mode">char</property> <property name="selectable">True</property> + <property name="max_width_chars">52</property> </object> <packing> <property name="left_attach">0</property> @@ -363,7 +373,7 @@ <property name="wrap">True</property> <property name="wrap_mode">char</property> <property name="selectable">True</property> - <property name="max_width_chars">42</property> + <property name="max_width_chars">52</property> </object> <packing> <property name="left_attach">0</property> ===================================== gajim/data/gui/server_info.ui ===================================== @@ -0,0 +1,370 @@ +<?xml version="1.0" encoding="UTF-8"?> + +<interface> + <requires lib="gtk+" version="3.20"/> + <object class="GtkNotebook" id="server_info_notebook"> + <property name="visible">True</property> + <property name="can_focus">True</property> + <child> + <object class="GtkGrid" id="server"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="row_spacing">6</property> + <property name="column_spacing">12</property> + <child> + <object class="GtkLabel"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="halign">end</property> + <property name="label" translatable="yes">Hostname</property> + <style> + <class name="dim-label"/> + </style> + </object> + <packing> + <property name="left_attach">0</property> + <property name="top_attach">0</property> + </packing> + </child> + <child> + <object class="GtkLabel" id="server_hostname"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="halign">start</property> + <property name="selectable">True</property> + <property name="ellipsize">end</property> + <property name="single_line_mode">True</property> + <property name="xalign">0</property> + </object> + <packing> + <property name="left_attach">1</property> + <property name="top_attach">0</property> + </packing> + </child> + <child> + <object class="GtkLabel"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="halign">end</property> + <property name="label" translatable="yes">Server Software</property> + <style> + <class name="dim-label"/> + </style> + </object> + <packing> + <property name="left_attach">0</property> + <property name="top_attach">1</property> + </packing> + </child> + <child> + <object class="GtkLabel" id="server_software"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="halign">start</property> + <property name="selectable">True</property> + <property name="ellipsize">end</property> + <property name="single_line_mode">True</property> + <property name="xalign">0</property> + </object> + <packing> + <property name="left_attach">1</property> + <property name="top_attach">1</property> + </packing> + </child> + <child> + <object class="GtkLabel"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="halign">end</property> + <property name="label" translatable="yes">Server Uptime</property> + <style> + <class name="dim-label"/> + </style> + </object> + <packing> + <property name="left_attach">0</property> + <property name="top_attach">2</property> + </packing> + </child> + <child> + <object class="GtkLabel" id="server_uptime"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="halign">start</property> + <property name="selectable">True</property> + <property name="ellipsize">end</property> + <property name="single_line_mode">True</property> + </object> + <packing> + <property name="left_attach">1</property> + <property name="top_attach">2</property> + </packing> + </child> + <child> + <object class="GtkBox" id="spacer1"> + <property name="width_request">160</property> + <property name="visible">True</property> + <property name="can_focus">False</property> + <child> + <placeholder/> + </child> + </object> + <packing> + <property name="left_attach">0</property> + <property name="top_attach">3</property> + </packing> + </child> + <child> + <placeholder/> + </child> + <style> + <class name="margin-18"/> + </style> + </object> + </child> + <child type="tab"> + <object class="GtkLabel"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="label" translatable="yes">Server</property> + </object> + <packing> + <property name="tab_fill">False</property> + </packing> + </child> + <child> + <object class="GtkGrid"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="row_spacing">6</property> + <property name="column_spacing">12</property> + <child> + <object class="GtkLabel"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="halign">end</property> + <property name="label" translatable="yes">Type</property> + <style> + <class name="dim-label"/> + </style> + </object> + <packing> + <property name="left_attach">0</property> + <property name="top_attach">0</property> + </packing> + </child> + <child> + <object class="GtkLabel" id="connection_type"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="halign">start</property> + <property name="single_line_mode">True</property> + <property name="xalign">0</property> + </object> + <packing> + <property name="left_attach">1</property> + <property name="top_attach">0</property> + </packing> + </child> + <child> + <object class="GtkLabel"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="halign">end</property> + <property name="label" translatable="yes">Security</property> + <style> + <class name="dim-label"/> + </style> + </object> + <packing> + <property name="left_attach">0</property> + <property name="top_attach">1</property> + </packing> + </child> + <child> + <object class="GtkLabel" id="connection_security"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="halign">start</property> + <property name="label">-</property> + <property name="single_line_mode">True</property> + <property name="xalign">0</property> + </object> + <packing> + <property name="left_attach">1</property> + <property name="top_attach">1</property> + </packing> + </child> + <child> + <object class="GtkLabel"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="halign">end</property> + <property name="label" translatable="yes">Cipher</property> + <style> + <class name="dim-label"/> + </style> + </object> + <packing> + <property name="left_attach">0</property> + <property name="top_attach">2</property> + </packing> + </child> + <child> + <object class="GtkLabel" id="connection_cipher"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="halign">start</property> + <property name="label">-</property> + <property name="selectable">True</property> + <property name="xalign">0</property> + </object> + <packing> + <property name="left_attach">1</property> + <property name="top_attach">2</property> + </packing> + </child> + <child> + <object class="GtkLabel" id="connection_proxy_header"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="halign">end</property> + <property name="label" translatable="yes">Proxy</property> + <style> + <class name="dim-label"/> + </style> + </object> + <packing> + <property name="left_attach">0</property> + <property name="top_attach">3</property> + </packing> + </child> + <child> + <object class="GtkLabel" id="connection_proxy"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="halign">start</property> + <property name="label" translatable="yes">No proxy used</property> + <property name="xalign">0</property> + </object> + <packing> + <property name="left_attach">1</property> + <property name="top_attach">3</property> + </packing> + </child> + <child> + <object class="GtkButton" id="cert_button"> + <property name="label" translatable="yes">View _Certificate</property> + <property name="visible">True</property> + <property name="can_focus">True</property> + <property name="receives_default">True</property> + <property name="halign">center</property> + <property name="margin_top">12</property> + <property name="use_underline">True</property> + <signal name="clicked" handler="_on_cert_button_clicked" swapped="no"/> + </object> + <packing> + <property name="left_attach">0</property> + <property name="top_attach">4</property> + <property name="width">2</property> + </packing> + </child> + <child> + <object class="GtkBox" id="spacer2"> + <property name="width_request">160</property> + <property name="visible">True</property> + <property name="can_focus">False</property> + <child> + <placeholder/> + </child> + </object> + <packing> + <property name="left_attach">0</property> + <property name="top_attach">5</property> + </packing> + </child> + <child> + <placeholder/> + </child> + <style> + <class name="margin-18"/> + </style> + </object> + <packing> + <property name="position">1</property> + </packing> + </child> + <child type="tab"> + <object class="GtkLabel"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="label" translatable="yes">Connection</property> + </object> + <packing> + <property name="position">1</property> + <property name="tab_fill">False</property> + </packing> + </child> + <child> + <object class="GtkBox" id="features"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="border_width">18</property> + <property name="orientation">vertical</property> + <property name="spacing">12</property> + <child> + <object class="GtkScrolledWindow" id="features_scrolled"> + <property name="visible">True</property> + <property name="can_focus">True</property> + <property name="hscrollbar_policy">never</property> + <property name="max_content_height">500</property> + <child> + <placeholder/> + </child> + </object> + <packing> + <property name="expand">True</property> + <property name="fill">True</property> + <property name="position">0</property> + </packing> + </child> + <child> + <object class="GtkButton" id="clipboard_button"> + <property name="visible">True</property> + <property name="can_focus">True</property> + <property name="receives_default">True</property> + <property name="tooltip_text" translatable="yes">Copy info to clipboard</property> + <property name="halign">end</property> + <signal name="clicked" handler="_on_clipboard_button_clicked" swapped="no"/> + <child> + <object class="GtkImage"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="icon_name">edit-copy-symbolic</property> + </object> + </child> + </object> + <packing> + <property name="expand">False</property> + <property name="fill">True</property> + <property name="position">1</property> + </packing> + </child> + </object> + <packing> + <property name="position">2</property> + </packing> + </child> + <child type="tab"> + <object class="GtkLabel"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="label" translatable="yes">Features</property> + </object> + <packing> + <property name="position">2</property> + <property name="tab_fill">False</property> + </packing> + </child> + </object> +</interface> ===================================== gajim/data/style/gajim.css ===================================== @@ -66,11 +66,11 @@ popover#EmoticonPopover { padding: 5px; background-color: @theme_unfocused_base_ #FinishedLabel { font-size: 14px; font-weight: bold } /* Server Info */ -#ServerInfoGrid > list { border: 1px solid; border-color: @borders; } -#ServerInfoGrid > list > row:first-child { border-top: 1px solid; border-color: @borders; } -#ServerInfoGrid > list > row { padding: 10px 20px 10px 10px; } -#ServerInfoGrid > list > label { padding:10px; color: @insensitive_fg_color; font-weight: bold; } -#ServerInfoGrid > list > row.activatable:active { box-shadow: none; } +#ServerInfo { border: 1px solid; border-color: @borders; } +#ServerInfo > row:first-child { border-top: 1px solid; border-color: @borders; } +#ServerInfo > row { padding: 10px 20px 10px 10px; } +#ServerInfo > label { padding:10px; color: @insensitive_fg_color; font-weight: bold; } +#ServerInfo > row.activatable:active { box-shadow: none; } /* Features Info */ #FeaturesInfoGrid > list { border: 1px solid; border-color: @borders; } @@ -225,6 +225,7 @@ list.settings > row > box { .margin-12 { margin: 12px; } .margin-18 { margin: 18px; } .padding-18 { margin: 18px; } +.padding-left30 { padding-left: 30px; } /* Treeview */ treeview.space { padding: 6px; } ===================================== gajim/gtk/dialogs.py ===================================== @@ -468,6 +468,7 @@ class CertificateDialog(Gtk.ApplicationWindow): self._ui = get_builder('certificate_dialog.ui') self.add(self._ui.certificate_box) + self.account = account self._clipboard = Gtk.Clipboard.get(Gdk.SELECTION_CLIPBOARD) # Get data for labels and copy button @@ -483,10 +484,10 @@ class CertificateDialog(Gtk.ApplicationWindow): self._ib_organization = issuer.organizationName or '' self._ib_org_unit = issuer.organizationalUnitName or '' issued = datetime.strptime(cert.get_notBefore().decode('ascii'), - '%Y%m%d%H%M%SZ') + '%Y%m%d%H%M%SZ') self._issued = issued.strftime('%B %d, %Y, %H:%M:%S %z') expires = datetime.strptime(cert.get_notAfter().decode('ascii'), - '%Y%m%d%H%M%SZ') + '%Y%m%d%H%M%SZ') self._expires = expires.strftime('%B %d, %Y, %H:%M:%S %z') self._sha1 = cert.digest('sha1').decode('utf-8') self._sha256 = cert.digest('sha256').decode('utf-8') ===================================== gajim/gtk/server_info.py ===================================== @@ -20,85 +20,68 @@ import nbxmpp from nbxmpp.util import is_error_result from gi.repository import Gtk from gi.repository import Gdk -from gi.repository import Pango from gajim.common import app from gajim.common import ged from gajim.common.i18n import _ +from gajim.gtk.dialogs import CertificateDialog from gajim.gtk.util import ensure_not_destroyed +from gajim.gtk.util import get_builder -log = logging.getLogger('gajim.gtk.serverinfo') +log = logging.getLogger('gajim.gtk.server_info') -class ServerInfoDialog(Gtk.Dialog): +class ServerInfo(Gtk.ApplicationWindow): def __init__(self, account): - super().__init__(title=_('Server Info'), - transient_for=None, - destroy_with_parent=True) + Gtk.ApplicationWindow.__init__(self) + self.set_name('ServerInfo') + self.set_application(app.app) + self.set_position(Gtk.WindowPosition.CENTER) + self.set_default_size(400, 600) + self.set_show_menubar(False) + self.set_title(_('Server Info')) self.account = account self._destroyed = False - self.set_transient_for(app.interface.roster.window) - self.set_resizable(True) - self.set_size_request(300, 500) - grid = Gtk.Grid() - grid.set_name('ServerInfoGrid') - grid.set_row_spacing(10) - grid.set_hexpand(True) + self._ui = get_builder('server_info.ui') + self.add(self._ui.server_info_notebook) - self.info_listbox = Gtk.ListBox() - self.info_listbox.set_selection_mode(Gtk.SelectionMode.NONE) - self.info_listbox.set_header_func(self.header_func, 'Information') - grid.attach(self.info_listbox, 0, 0, 1, 1) - - self.feature_listbox = Gtk.ListBox() - self.feature_listbox.set_selection_mode(Gtk.SelectionMode.NONE) - self.feature_listbox.set_header_func(self.header_func, 'Features') - grid.attach(self.feature_listbox, 0, 1, 1, 1) - - self.clipboard = Gtk.Clipboard.get(Gdk.SELECTION_CLIPBOARD) - clipboard_button = Gtk.Button(halign=Gtk.Align.END) - clp_image = Gtk.Image.new_from_icon_name('edit-copy-symbolic', - Gtk.IconSize.BUTTON) - clipboard_button.set_image(clp_image) - clipboard_button.set_tooltip_text(_('Copy info to clipboard')) - clipboard_button.connect('clicked', self.on_clipboard_button_clicked) - - box = self.get_content_area() - scrolled = Gtk.ScrolledWindow() - scrolled.set_max_content_height(500) - scrolled.set_policy(Gtk.PolicyType.NEVER, Gtk.PolicyType.AUTOMATIC) - scrolled.add(grid) - box.pack_start(scrolled, True, True, 0) - box.pack_start(clipboard_button, False, True, 0) - box.set_property('margin', 12) - box.set_spacing(18) - - self.connect('response', self.on_response) self.connect('destroy', self.on_destroy) + self.connect('key-press-event', self._on_key_press) + self._ui.connect_signals(self) app.ged.register_event_handler('server-disco-received', ged.GUI1, self._server_disco_received) self.version = '' - self.uptime = '' self.hostname = app.get_hostname_from_account(account) + self._ui.server_hostname.set_text(self.hostname) con = app.connections[account] con.get_module('SoftwareVersion').request_software_version( self.hostname, callback=self._software_version_received) self.request_last_activity() + self.cert = con.connection.Connection.ssl_certificate + self._add_connection_info() + + self.feature_listbox = Gtk.ListBox() + self.feature_listbox.set_name('ServerInfo') + self.feature_listbox.set_selection_mode(Gtk.SelectionMode.NONE) + self.feature_listbox.set_header_func(self.header_func, 'Features') + self._ui.features_scrolled.add(self.feature_listbox) for feature in self.get_features(): self.add_feature(feature) - - for info in self.get_infos(): - self.add_info(info) + self.clipboard = Gtk.Clipboard.get(Gdk.SELECTION_CLIPBOARD) self.show_all() + def _on_key_press(self, _widget, event): + if event.keyval == Gdk.KEY_Escape: + self.destroy() + @staticmethod def header_func(row, before, user_data): if before: @@ -108,12 +91,48 @@ class ServerInfoDialog(Gtk.Dialog): label.set_halign(Gtk.Align.START) row.set_header(label) - @staticmethod - def update(func, listbox): - for index, item in enumerate(func()): - row = listbox.get_row_at_index(index) - row.get_child().update(item) - row.set_tooltip_text(row.get_child().tooltip) + def _add_connection_info(self): + ssl_con = app.connections[self.account].connection.get_ssl_connection() + ssl_version = None + cipher_name = None + if ssl_con is not None: + ssl_version = ssl_con.get_cipher_version() + cipher_name = ssl_con.get_cipher_name() + + host, proxy = app.connections[self.account].get_connection_info() + con_type = host['type'] + + # Connection type + self._ui.connection_type.set_text(con_type.upper()) + if con_type == 'plain': + self._ui.conection_type.get_style_context().add_class( + 'error-color') + + # Connection security + if ssl_version is not None: + self._ui.connection_security.set_text(ssl_version) + + # Connection cipher + if cipher_name is not None: + self._ui.connection_cipher.set_text(cipher_name) + + # Connection proxy + if proxy: + if proxy['type'] == 'bosh': + self._ui.connection_proxy_header.set_text(_('BOSH')) + self._ui.connection_proxy.set_text(proxy['bosh_uri']) + if proxy['type'] in ['http', 'socks5'] or proxy['bosh_useproxy']: + self._ui.connection_proxy.set_text( + proxy['host'] + ':' + proxy['port']) + + self._ui.cert_button.set_sensitive(self.cert) + + def _on_cert_button_clicked(self, button_): + window = app.get_app_window(CertificateDialog, self.account) + if window is None: + CertificateDialog(self, self.account, self.cert) + else: + window.present() def request_last_activity(self): if not app.account_is_connected(self.account): @@ -123,7 +142,7 @@ class ServerInfoDialog(Gtk.Dialog): con.connection.SendAndCallForResponse(iq, self._on_last_activity) def _on_last_activity(self, stanza): - if 'server_info' not in app.interface.instances[self.account]: + if self._destroyed: # Window got closed in the meantime return if not nbxmpp.isResultNode(stanza): @@ -141,9 +160,9 @@ class ServerInfoDialog(Gtk.Dialog): hours = 0 if seconds >= 3600: hours = delta.seconds // 3600 - self.uptime = _('%(days)s days, %(hours)s hours') % { + uptime = _('%(days)s days, %(hours)s hours') % { 'days': delta.days, 'hours': hours} - self.update(self.get_infos, self.info_listbox) + self._ui.server_uptime.set_text(uptime) @ensure_not_destroyed def _software_version_received(self, result): @@ -151,7 +170,14 @@ class ServerInfoDialog(Gtk.Dialog): self.version = _('Unknown') else: self.version = '%s %s' % (result.name, result.version) - self.update(self.get_infos, self.info_listbox) + self._ui.server_software.set_text(self.version) + + @staticmethod + def update(func, listbox): + for index, item in enumerate(func()): + row = listbox.get_row_at_index(index) + row.get_child().update(item) + row.set_tooltip_text(row.get_child().tooltip) def _server_disco_received(self, obj): self.update(self.get_features, self.feature_listbox) @@ -209,18 +235,8 @@ class ServerInfoDialog(Gtk.Dialog): con.get_module('Bookmarks').conversion) ] - def add_info(self, info): - self.info_listbox.add(ServerInfoItem(info)) - - def get_infos(self): - Info = namedtuple('Info', ['name', 'value', 'tooltip']) - return [ - Info(_('Hostname'), self.hostname, None), - Info(_('Server Software'), self.version, None), - Info(_('Server Uptime'), self.uptime, None)] - - def on_clipboard_button_clicked(self, widget): - server_software = 'Server Software: %s\n' % self.get_infos()[1].value + def _on_clipboard_button_clicked(self, widget): + server_software = 'Server Software: %s\n' % self.version server_features = '' for feature in self.get_features(): @@ -232,18 +248,14 @@ class ServerInfoDialog(Gtk.Dialog): tooltip = '(%s)' % feature.tooltip else: tooltip = '' - server_features += '%s: %s %s\n' % (feature.name, available, tooltip) + server_features += '%s: %s %s\n' % ( + feature.name, available, tooltip) clipboard_text = server_software + server_features self.clipboard.set_text(clipboard_text, -1) - def on_response(self, dialog, response): - if response == Gtk.ResponseType.OK: - self.destroy() - def on_destroy(self, *args): self._destroyed = True - del app.interface.instances[self.account]['server_info'] app.ged.remove_event_handler('server-disco-received', ged.GUI1, self._server_disco_received) @@ -274,7 +286,7 @@ class FeatureItem(Gtk.Grid): elif enabled is False: self.icon.set_from_icon_name('dialog-warning-symbolic', Gtk.IconSize.MENU) - self.tooltip += _('\nDisabled in config') + self.tooltip += _('\nDisabled in preferences') self.icon.get_style_context().add_class('warning-color') else: self.icon.set_from_icon_name('emblem-ok-symbolic', @@ -284,28 +296,3 @@ class FeatureItem(Gtk.Grid): def update(self, feature): self.tooltip = feature.tooltip self.set_feature(feature.available, feature.enabled) - - -class ServerInfoItem(Gtk.Grid): - def __init__(self, info): - super().__init__() - self.tooltip = info.tooltip - self.insert_column(0) - - self.info = Gtk.Label(label=info.name) - self.info.set_halign(Gtk.Align.START) - self.info.set_xalign(0) - self.info.set_size_request(160, -1) - self.value = Gtk.Label(label=info.value) - self.value.set_halign(Gtk.Align.START) - self.value.set_ellipsize(Pango.EllipsizeMode.END) - self.value.set_width_chars(20) - self.value.set_xalign(0) - self.value.set_hexpand(True) - self.value.set_selectable(True) - - self.add(self.info) - self.add(self.value) - - def update(self, info): - self.value.set_text(info.value) ===================================== gajim/roster_window.py ===================================== @@ -72,8 +72,8 @@ from gajim.gtk.dialogs import InputDialog from gajim.gtk.dialogs import WarningDialog from gajim.gtk.dialogs import InformationDialog from gajim.gtk.dialogs import NonModalConfirmationDialog -from gajim.gtk.dialogs import CertificateDialog from gajim.gtk.dialogs import InvitationReceivedDialog +from gajim.gtk.server_info import ServerInfo from gajim.gtk.single_message import SingleMessageWindow from gajim.gtk.add_contact import AddNewContactWindow from gajim.gtk.account_wizard import AccountCreationWizard @@ -3531,10 +3531,12 @@ class RosterWindow: jid = jid + '/' + resource AdHocCommand(account, jid) - def on_view_certificate(self, widget, account): - con = app.connections[account] - cert = con.connection.Connection.ssl_certificate - CertificateDialog(self.window, account, cert) + def on_view_server_info(self, _widget, account): + window = app.get_app_window(ServerInfo, account) + if window is None: + ServerInfo(account) + else: + window.present() def on_roster_window_focus_in_event(self, widget, event): # roster received focus, so if we had urgency REMOVE IT @@ -4761,8 +4763,8 @@ class RosterWindow: 'service_discovery_menuitem') execute_command_menuitem = xml.get_object( 'execute_command_menuitem') - view_certificate_menuitem = xml.get_object( - 'view_certificate_menuitem') + view_server_info_menuitem = xml.get_object( + 'view_server_info_menuitem') edit_account_menuitem = xml.get_object('edit_account_menuitem') sub_menu = Gtk.Menu() status_menuitem.set_submenu(sub_menu) @@ -4840,14 +4842,14 @@ class RosterWindow: account=account) # Fake contact execute_command_menuitem.connect('activate', self.on_execute_command, contact, account) - view_certificate_menuitem.connect('activate', - self.on_view_certificate, account) + view_server_info_menuitem.connect('activate', + self.on_view_server_info, account) # make some items insensitive if account is offline if not app.account_is_connected(account): for widget in (add_contact_menuitem, service_discovery_menuitem, - execute_command_menuitem, - view_certificate_menuitem, pep_menuitem): + execute_command_menuitem, view_server_info_menuitem, + pep_menuitem): widget.set_sensitive(False) else: xml = get_builder('zeroconf_context_menu.ui') View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/8936a07e46a0e54bb4a43dcb6cd9072862… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/8936a07e46a0e54bb4a43dcb6cd9072862… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Bookmarks: Register event handler correctly
by Philipp Hörist 28 Sep '19

28 Sep '19

Philipp Hörist pushed to branch master at gajim / gajim Commits: 0c84072e by Philipp Hörist at 2019-09-28T23:33:52Z Bookmarks: Register event handler correctly - - - - - 1 changed file: - gajim/common/modules/bookmarks.py Changes: ===================================== gajim/common/modules/bookmarks.py ===================================== @@ -43,6 +43,7 @@ class Bookmarks(BaseModule): def __init__(self, con): BaseModule.__init__(self, con) self._register_pubsub_handler(self._bookmark_event_received) + self._register_pubsub_handler(self._bookmark_2_event_received) self._conversion = False self._conversion_2 = False self._bookmarks = [] @@ -108,7 +109,7 @@ class Bookmarks(BaseModule): NetworkEvent('bookmarks-received', account=self._account)) @event_node(nbxmpp.NS_BOOKMARKS_2) - def _bookmark_event_received(self, _con, stanza, properties): + def _bookmark_2_event_received(self, _con, stanza, properties): if not properties.is_self_message: self._log.warning('%s has an open access bookmarks node', properties.jid) View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/0c84072ea78a2f0bd6d432cebe152fce6d… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/0c84072ea78a2f0bd6d432cebe152fce6d… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Add Bookmarks 2 (XEP-0402) support
by Philipp Hörist 28 Sep '19

28 Sep '19

Philipp Hörist pushed to branch master at gajim / gajim Commits: 3d381a98 by Philipp Hörist at 2019-09-28T23:16:57Z Add Bookmarks 2 (XEP-0402) support - - - - - 6 changed files: - gajim/common/app.py - gajim/common/helpers.py - gajim/common/modules/bookmarks.py - gajim/common/modules/discovery.py - gajim/common/modules/muc.py - gajim/gtk/bookmarks.py Changes: ===================================== gajim/common/app.py ===================================== @@ -181,7 +181,6 @@ gajim_common_features = [ nbxmpp.NS_JINGLE_BYTESTREAM, nbxmpp.NS_JINGLE_IBB, nbxmpp.NS_AVATAR_METADATA + '+notify', - nbxmpp.NS_BOOKMARKS + '+notify', ] # Optional features gajim supports per account ===================================== gajim/common/helpers.py ===================================== @@ -1087,6 +1087,10 @@ def update_optional_features(account=None): features.append(nbxmpp.NS_NICK + '+notify') if app.config.get_per('accounts', account_, 'subscribe_location'): features.append(nbxmpp.NS_LOCATION + '+notify') + if app.connections[account_].get_module('Bookmarks').using_bookmark_2: + features.append(nbxmpp.NS_BOOKMARKS_2 + '+notify') + elif app.connections[account_].get_module('Bookmarks').using_bookmark_1: + features.append(nbxmpp.NS_BOOKMARKS + '+notify') if not app.config.get('ignore_incoming_xhtml'): features.append(nbxmpp.NS_XHTML_IM) if app.config.get_per('accounts', account_, 'answer_receipts'): ===================================== gajim/common/modules/bookmarks.py ===================================== @@ -37,12 +37,14 @@ class Bookmarks(BaseModule): _nbxmpp_methods = [ 'request_bookmarks', 'store_bookmarks', + 'retract_bookmark', ] def __init__(self, con): BaseModule.__init__(self, con) self._register_pubsub_handler(self._bookmark_event_received) self._conversion = False + self._conversion_2 = False self._bookmarks = [] self._join_timeouts = [] self._request_in_progress = False @@ -51,6 +53,10 @@ class Bookmarks(BaseModule): def conversion(self): return self._conversion + @property + def conversion_2(self): + return self._conversion_2 + @property def bookmarks(self): return self._bookmarks @@ -59,6 +65,14 @@ class Bookmarks(BaseModule): def bookmarks(self, value): self._bookmarks = value + @property + def using_bookmark_1(self): + return self._pubsub_support() and self.conversion + + @property + def using_bookmark_2(self): + return self._pubsub_support() and self.conversion_2 + @event_node(nbxmpp.NS_BOOKMARKS) def _bookmark_event_received(self, _con, stanza, properties): if properties.pubsub_event.retracted: @@ -66,7 +80,7 @@ class Bookmarks(BaseModule): return bookmarks = properties.pubsub_event.data - if properties.pubsub_event.deleted: + if properties.pubsub_event.deleted or properties.pubsub_event.purged: self._log.info('Bookmark node deleted') bookmarks = [] @@ -80,7 +94,7 @@ class Bookmarks(BaseModule): properties.jid) return - if not self._pubsub_support() or not self.conversion: + if not self.using_bookmark_1: return if self._request_in_progress: @@ -93,11 +107,60 @@ class Bookmarks(BaseModule): app.nec.push_incoming_event( NetworkEvent('bookmarks-received', account=self._account)) - def pass_disco(self, info): - if nbxmpp.NS_BOOKMARK_CONVERSION not in info.features: + @event_node(nbxmpp.NS_BOOKMARKS_2) + def _bookmark_event_received(self, _con, stanza, properties): + if not properties.is_self_message: + self._log.warning('%s has an open access bookmarks node', + properties.jid) + return + + if not self.using_bookmark_2: + return + + if self._request_in_progress: + self._log.info('Ignore update, pubsub request in progress') return - self._conversion = True - self._log.info('Discovered Bookmarks Conversion: %s', info.jid) + + old_bookmarks = self._convert_to_set(self._bookmarks) + + if properties.pubsub_event.deleted or properties.pubsub_event.purged: + self._log.info('Bookmark node deleted') + self._bookmarks = [] + + elif properties.pubsub_event.retracted: + jid = properties.pubsub_event.id + self._log.info('Retract: %s', jid) + bookmark = self.get_bookmark_from_jid(jid) + if bookmark is not None: + try: + self._bookmarks.remove(bookmark) + except KeyError: + pass + + elif properties.pubsub_event.data is None: + self._log.warning('Invalid bookmark data') + self._log.warning(stanza) + return + + else: + new_bookmark = properties.pubsub_event.data + old_bookmark = self.get_bookmark_from_jid(new_bookmark.jid) + if old_bookmark is not None: + self._bookmarks.remove(old_bookmark) + self._bookmarks.append(new_bookmark) + + self._act_on_changed_bookmarks(old_bookmarks) + app.nec.push_incoming_event( + NetworkEvent('bookmarks-received', account=self._account)) + + def pass_disco(self, info): + if nbxmpp.NS_BOOKMARKS_COMPAT in info.features: + self._conversion_2 = True + self._log.info('Discovered Bookmarks Conversion 2: %s', info.jid) + + elif nbxmpp.NS_BOOKMARK_CONVERSION in info.features: + self._conversion = True + self._log.info('Discovered Bookmarks Conversion: %s', info.jid) def _act_on_changed_bookmarks(self, old_bookmarks): new_bookmarks = self._convert_to_set(self._bookmarks) @@ -142,8 +205,11 @@ class Bookmarks(BaseModule): self._request_in_progress = True type_ = BookmarkStoreType.PRIVATE - if self._pubsub_support() and self.conversion: - type_ = BookmarkStoreType.PUBSUB + if self._pubsub_support(): + if self.conversion: + type_ = BookmarkStoreType.PUBSUB_BOOKMARK_1 + if self._conversion_2: + type_ = BookmarkStoreType.PUBSUB_BOOKMARK_2 self._nbxmpp('Bookmarks').request_bookmarks( type_, callback=self._bookmarks_received) @@ -159,15 +225,48 @@ class Bookmarks(BaseModule): app.nec.push_incoming_event( NetworkEvent('bookmarks-received', account=self._account)) - def store_bookmarks(self): + def store_difference(self, bookmarks): + if self.using_bookmark_2: + retract, add_or_modify = self._determine_changed_bookmarks( + bookmarks, self._bookmarks) + + for bookmark in retract: + self.remove(str(bookmark.jid)) + + if add_or_modify: + self.store_bookmarks(add_or_modify) + self._bookmarks = bookmarks + + else: + self._bookmarks = bookmarks + self.store_bookmarks() + + def store_bookmarks(self, bookmarks=None): if not app.account_is_connected(self._account): return type_ = BookmarkStoreType.PRIVATE - if self._pubsub_support() and self.conversion: - type_ = BookmarkStoreType.PUBSUB + if self._pubsub_support(): + if self.conversion: + type_ = BookmarkStoreType.PUBSUB_BOOKMARK_1 + if self.conversion_2: + type_ = BookmarkStoreType.PUBSUB_BOOKMARK_2 + + self._nbxmpp('Bookmarks').store_bookmarks(bookmarks or self._bookmarks, + type_) + + app.nec.push_incoming_event( + NetworkEvent('bookmarks-received', account=self._account)) + + def store_bookmark(self, bookmark): + if not app.account_is_connected(self._account): + return + + if not self.using_bookmark_2: + return - self._nbxmpp('Bookmarks').store_bookmarks(self._bookmarks, type_) + self._nbxmpp('Bookmarks').store_bookmarks( + [bookmark], BookmarkStoreType.PUBSUB_BOOKMARK_2) app.nec.push_incoming_event( NetworkEvent('bookmarks-received', account=self._account)) @@ -199,19 +298,35 @@ class Bookmarks(BaseModule): def modify(self, jid: str, **kwargs: Dict[str, str]) -> None: bookmark = self.get_bookmark_from_jid(jid) if bookmark is None: - bookmark = BookmarkData(jid=jid, **kwargs) - self._bookmarks.append(bookmark) - self._log.info('Add new bookmark: %s', bookmark) + return + + new_bookmark = bookmark._replace(**kwargs) + if new_bookmark == bookmark: + # No change happened + return + self._log.info('Modify bookmark: %s %s', jid, kwargs) + self._bookmarks.remove(bookmark) + self._bookmarks.append(new_bookmark) + + if self.using_bookmark_2: + self.store_bookmark(new_bookmark) else: - modified_bookmark = bookmark._replace(**kwargs) - if modified_bookmark == bookmark: - # No change happened - return - self._log.info('Modify bookmark: %s %s', jid, kwargs) - self._bookmarks.remove(bookmark) - self._bookmarks.append(modified_bookmark) + self.store_bookmarks() + + def add_or_modify(self, jid: str, **kwargs: Dict[str, str]) -> None: + bookmark = self.get_bookmark_from_jid(jid) + if bookmark is not None: + self.modify(jid, **kwargs) + return - self.store_bookmarks() + new_bookmark = BookmarkData(jid=jid, **kwargs) + self._bookmarks.append(new_bookmark) + self._log.info('Add new bookmark: %s', new_bookmark) + + if self.using_bookmark_2: + self.store_bookmark(new_bookmark) + else: + self.store_bookmarks() def remove(self, jid: str, publish: bool = True) -> None: bookmark = self.get_bookmark_from_jid(jid) @@ -219,7 +334,27 @@ class Bookmarks(BaseModule): return self._bookmarks.remove(bookmark) if publish: - self.store_bookmarks() + if self.using_bookmark_2: + self._nbxmpp('Bookmarks').retract_bookmark(jid) + else: + self.store_bookmarks() + + @staticmethod + def _determine_changed_bookmarks(new_bookmarks, old_bookmarks): + new_jids = [bookmark.jid for bookmark in new_bookmarks] + new_bookmarks = set(new_bookmarks) + old_bookmarks = set(old_bookmarks) + + retract = [] + add_or_modify = [] + changed_bookmarks = new_bookmarks.symmetric_difference(old_bookmarks) + + for bookmark in changed_bookmarks: + if bookmark.jid not in new_jids: + retract.append(bookmark) + if bookmark in new_bookmarks: + add_or_modify.append(bookmark) + return retract, add_or_modify def get_name_from_bookmark(self, jid: str) -> str: bookmark = self.get_bookmark_from_jid(jid) ===================================== gajim/common/modules/discovery.py ===================================== @@ -20,8 +20,9 @@ from nbxmpp.util import is_error_result from gajim.common import app from gajim.common.nec import NetworkIncomingEvent -from gajim.common.modules.base import BaseModule from gajim.common.nec import NetworkEvent +from gajim.common.modules.base import BaseModule +from gajim.common.helpers import update_optional_features class Discovery(BaseModule): @@ -106,6 +107,8 @@ class Discovery(BaseModule): if 'urn:xmpp:pep-vcard-conversion:0' in result.features: self._con.avatar_conversion = True + update_optional_features() + def discover_server_info(self): # Calling this method starts the connect_maschine() server = self._con.get_own_jid().getDomain() ===================================== gajim/common/modules/muc.py ===================================== @@ -688,10 +688,11 @@ class MUC(BaseModule): self._remove_rejoin_timeout(muc_data.jid) # We successfully joined a MUC, set add bookmark with autojoin - self._con.get_module('Bookmarks').modify(muc_data.jid, - autojoin=True, - password=muc_data.password, - nick=muc_data.nick) + self._con.get_module('Bookmarks').add_or_modify( + muc_data.jid, + autojoin=True, + password=muc_data.password, + nick=muc_data.nick) def _on_voice_request(self, _con, _stanza, properties): if not properties.is_voice_request: ===================================== gajim/gtk/bookmarks.py ===================================== @@ -19,6 +19,7 @@ from gi.repository import Gtk from gi.repository import Gdk from nbxmpp.structs import BookmarkData from nbxmpp.protocol import validate_resourcepart +from nbxmpp.protocol import JID from gajim.common import app from gajim.common.helpers import validate_jid @@ -142,7 +143,8 @@ class Bookmarks(Gtk.ApplicationWindow): for row in self._ui.bookmarks_store: if not row[Column.ADDRESS]: continue - bookmark = BookmarkData(jid=row[Column.ADDRESS], + + bookmark = BookmarkData(jid=JID(row[Column.ADDRESS]), name=row[Column.NAME], autojoin=row[Column.AUTOJOIN], password=row[Column.PASSWORD], @@ -150,8 +152,7 @@ class Bookmarks(Gtk.ApplicationWindow): bookmarks.append(bookmark) con = app.connections[self.account] - con.get_module('Bookmarks').bookmarks = bookmarks - con.get_module('Bookmarks').store_bookmarks() + con.get_module('Bookmarks').store_difference(bookmarks) self.destroy() @staticmethod View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/3d381a9863679ea97a0f33a5f6dbd2e9ee… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/3d381a9863679ea97a0f33a5f6dbd2e9ee… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/python-nbxmpp][master] Add Bookmarks 2 (XEP-0402) support
by Philipp Hörist 28 Sep '19

28 Sep '19

Philipp Hörist pushed to branch master at gajim / python-nbxmpp Commits: 550decfd by Philipp Hörist at 2019-09-28T23:12:58Z Add Bookmarks 2 (XEP-0402) support - - - - - 5 changed files: - nbxmpp/const.py - nbxmpp/modules/bookmarks.py - nbxmpp/modules/pubsub.py - nbxmpp/protocol.py - nbxmpp/structs.py Changes: ===================================== nbxmpp/const.py ===================================== @@ -310,7 +310,8 @@ Role._WEIGHTS = { class BookmarkStoreType(Enum): - PUBSUB = 'pubsub' + PUBSUB_BOOKMARK_2 = 'pubsub bookmark 2' + PUBSUB_BOOKMARK_1 = 'pubsub bookmark 1' PRIVATE = 'private' ===================================== nbxmpp/modules/bookmarks.py ===================================== @@ -18,6 +18,7 @@ import logging from nbxmpp.protocol import NS_BOOKMARKS +from nbxmpp.protocol import NS_BOOKMARKS_2 from nbxmpp.protocol import NS_PUBSUB_EVENT from nbxmpp.protocol import NS_PRIVATE from nbxmpp.protocol import isResultNode @@ -32,13 +33,31 @@ from nbxmpp.util import to_xs_boolean from nbxmpp.util import call_on_response from nbxmpp.util import callback from nbxmpp.util import raise_error +from nbxmpp.util import is_error_result from nbxmpp.modules.pubsub import get_pubsub_item +from nbxmpp.modules.pubsub import get_pubsub_items from nbxmpp.modules.pubsub import get_pubsub_request -from nbxmpp.modules.pubsub import get_bookmark_publish_options +from nbxmpp.modules.pubsub import get_publish_options + log = logging.getLogger('nbxmpp.m.bookmarks') +BOOKMARK_1_OPTIONS = { + 'pubsub#persist_items': 'true', + 'pubsub#access_model': 'whitelist', +} + +BOOKMARK_2_OPTIONS = { + 'pubsub#notify_delete': 'true', + 'pubsub#notify_retract': 'true', + 'pubsub#persist_items': 'true', + 'pubsub#max_items': '128', + 'pubsub#access_model': 'whitelist', + 'pubsub#send_last_published_item': 'never', +} + + class Bookmarks: def __init__(self, client): self._client = client @@ -47,8 +66,17 @@ class Bookmarks: callback=self._process_pubsub_bookmarks, ns=NS_PUBSUB_EVENT, priority=16), + StanzaHandler(name='message', + callback=self._process_pubsub_bookmarks2, + ns=NS_PUBSUB_EVENT, + priority=16), ] + self._bookmark_2_queue = {} + self._bookmark_1_queue = [] + self._node_configuration_in_progress = False + self._node_configuration_not_possible = False + def _process_pubsub_bookmarks(self, _con, _stanza, properties): if not properties.is_pubsub_event: return @@ -78,6 +106,30 @@ class Bookmarks: properties.pubsub_event = pubsub_event + def _process_pubsub_bookmarks2(self, _con, _stanza, properties): + if not properties.is_pubsub_event: + return + + if properties.pubsub_event.node != NS_BOOKMARKS_2: + return + + if properties.pubsub_event.deleted or properties.pubsub_event.retracted: + return + + item = properties.pubsub_event.item + if item is None: + return + + bookmark_item = self._parse_bookmarks2(item) + if bookmark_item is None: + return + + pubsub_event = properties.pubsub_event._replace(data=bookmark_item) + log.info('Received bookmark item from: %s', properties.jid) + log.info(bookmark_item) + + properties.pubsub_event = pubsub_event + @staticmethod def _parse_bookmarks(storage): bookmarks = [] @@ -111,6 +163,35 @@ class Bookmarks: return bookmarks + @staticmethod + def _parse_bookmarks2(item): + jid = item.getAttr('id') + if jid is None: + log.warning('No id attr found') + return + + try: + jid = JID(jid) + except Exception as error: + log.warning('Invalid JID: %s', error) + log.warning(item) + return + + conference = item.getTag('conference', namespace=NS_BOOKMARKS_2) + if conference is None: + log.warning('No conference node found') + log.warning(item) + return + + autojoin = conference.getAttr('autojoin') == 'true' + name = conference.getAttr('name') + nick = conference.getTagData('nick') + + return BookmarkData(jid=jid, + name=name or None, + autojoin=autojoin, + nick=nick or None) + @staticmethod def get_private_request(): iq = Iq(typ='get') @@ -121,10 +202,16 @@ class Bookmarks: @call_on_response('_bookmarks_received') def request_bookmarks(self, type_): jid = self._client.get_bound_jid().getBare() - if type_ == BookmarkStoreType.PUBSUB: + if type_ == BookmarkStoreType.PUBSUB_BOOKMARK_2: + log.info('Request bookmarks 2 (PubSub)') + request = get_pubsub_request(jid, NS_BOOKMARKS_2) + return request, {'type_': type_} + + if type_ == BookmarkStoreType.PUBSUB_BOOKMARK_1: log.info('Request bookmarks (PubSub)') request = get_pubsub_request(jid, NS_BOOKMARKS, max_items=1) return request, {'type_': type_} + if type_ == BookmarkStoreType.PRIVATE: log.info('Request bookmarks (Private Storage)') return self.get_private_request(), {'type_': type_} @@ -135,16 +222,27 @@ class Bookmarks: return raise_error(log.info, stanza) bookmarks = [] - if type_ == BookmarkStoreType.PUBSUB: + if type_ == BookmarkStoreType.PUBSUB_BOOKMARK_2: + items = get_pubsub_items(stanza, NS_BOOKMARKS_2) + if items is None: + return bookmarks + for item in items: + bookmark_item = self._parse_bookmarks2(item) + if bookmark_item is not None: + bookmarks.append(bookmark_item) + + elif type_ == BookmarkStoreType.PUBSUB_BOOKMARK_1: item = get_pubsub_item(stanza) - storage_node = item.getTag('storage', namespace=NS_BOOKMARKS) + if item is not None: + storage_node = item.getTag('storage', namespace=NS_BOOKMARKS) + if storage_node.getChildren(): + bookmarks = self._parse_bookmarks(storage_node) - if type_ == BookmarkStoreType.PRIVATE: + elif type_ == BookmarkStoreType.PRIVATE: query = stanza.getQuery() storage_node = query.getTag('storage', namespace=NS_BOOKMARKS) - - if storage_node.getChildren(): - bookmarks = self._parse_bookmarks(storage_node) + if storage_node.getChildren(): + bookmarks = self._parse_bookmarks(storage_node) from_ = stanza.getFrom() if from_ is None: @@ -169,22 +267,137 @@ class Bookmarks: conf_node.setTagData('password', bookmark.password) return storage_node + @staticmethod + def _build_conference_node(bookmark): + attrs = {'xmlns': NS_BOOKMARKS_2} + if bookmark.autojoin: + attrs['autojoin'] = 'true' + if bookmark.name: + attrs['name'] = bookmark.name + conference = Node(tag='conference', attrs=attrs) + if bookmark.nick: + conference.setTagData('nick', bookmark.nick) + return conference + def store_bookmarks(self, bookmarks, type_): - if type_ == BookmarkStoreType.PUBSUB: - self._store_with_pubsub(bookmarks) + if type_ == BookmarkStoreType.PUBSUB_BOOKMARK_2: + self._store_bookmark_2(bookmarks) + elif type_ == BookmarkStoreType.PUBSUB_BOOKMARK_1: + self._store_bookmark_1(bookmarks) elif type_ == BookmarkStoreType.PRIVATE: self._store_with_private(bookmarks) - def _store_with_pubsub(self, bookmarks): - log.info('Store Bookmarks (PubSub)') + def retract_bookmark(self, bookmark_jid): + log.info('Retract Bookmark: %s', bookmark_jid) + jid = self._client.get_bound_jid().getBare() + self._client.get_module('PubSub').retract(jid, + NS_BOOKMARKS_2, + str(bookmark_jid)) + + def _store_bookmark_1(self, bookmarks): + log.info('Store Bookmarks 1 (PubSub)') jid = self._client.get_bound_jid().getBare() + self._bookmark_1_queue = bookmarks item = self._build_storage_node(bookmarks) - options = get_bookmark_publish_options() - self._client.get_module('PubSub').publish(jid, - NS_BOOKMARKS, - item, - id_='current', - options=options) + options = get_publish_options(BOOKMARK_1_OPTIONS) + self._client.get_module('PubSub').publish( + jid, + NS_BOOKMARKS, + item, + id_='current', + options=options, + callback=self._on_store_bookmark_result, + user_data=NS_BOOKMARKS) + + def _store_bookmark_2(self, bookmarks): + if self._node_configuration_not_possible: + log.warning('Node configuration not possible') + return + + log.info('Store Bookmarks 2 (PubSub)') + jid = self._client.get_bound_jid().getBare() + for bookmark in bookmarks: + self._bookmark_2_queue[bookmark.jid] = bookmark + item = self._build_conference_node(bookmark) + options = get_publish_options(BOOKMARK_2_OPTIONS) + self._client.get_module('PubSub').publish( + jid, + NS_BOOKMARKS_2, + item, + id_=str(bookmark.jid), + options=options, + callback=self._on_store_bookmark_result, + user_data=NS_BOOKMARKS_2) + + def _on_store_bookmark_result(self, result, node): + if not is_error_result(result): + self._bookmark_1_queue = [] + self._bookmark_2_queue.pop(result.id, None) + return + + if (result.condition == 'conflict' and + result.app_condition == 'precondition-not-met'): + if self._node_configuration_in_progress: + return + + self._node_configuration_in_progress = True + jid = self._client.get_bound_jid().getBare() + self._client.get_module('PubSub').get_node_configuration( + jid, + node, + callback=self._on_node_configuration_received) + + else: + self._bookmark_1_queue = [] + self._bookmark_2_queue.pop(result.id, None) + log.warning(result) + + def _on_node_configuration_received(self, result): + if is_error_result(result): + log.warning(result) + self._bookmark_1_queue = [] + self._bookmark_2_queue.clear() + return + + if result.node == NS_BOOKMARKS: + config = BOOKMARK_1_OPTIONS + else: + config = BOOKMARK_2_OPTIONS + self._apply_config(result.form, config) + self._client.get_module('PubSub').set_node_configuration( + result.jid, + result.node, + result.form, + callback=self._on_node_configuration_finished) + + def _on_node_configuration_finished(self, result): + self._node_configuration_in_progress = False + if is_error_result(result): + log.warning(result) + self._bookmark_2_queue.clear() + self._bookmark_1_queue = [] + self._node_configuration_not_possible = True + return + + log.info('Republish bookmarks') + if self._bookmark_2_queue: + bookmarks = self._bookmark_2_queue.copy() + self._bookmark_2_queue.clear() + self._store_bookmark_2(bookmarks.values()) + else: + bookmarks = self._bookmark_1_queue.copy() + self._bookmark_1_queue.clear() + self._store_bookmark_1(bookmarks) + + @staticmethod + def _apply_config(form, config): + for var, value in config.items(): + try: + field = form[var] + except KeyError: + pass + else: + field.value = value @call_on_response('_on_private_store_result') def _store_with_private(self, bookmarks): ===================================== nbxmpp/modules/pubsub.py ===================================== @@ -20,6 +20,8 @@ import logging from nbxmpp.protocol import NS_PUBSUB from nbxmpp.protocol import NS_PUBSUB_EVENT from nbxmpp.protocol import NS_PUBSUB_PUBLISH_OPTIONS +from nbxmpp.protocol import NS_PUBSUB_OWNER +from nbxmpp.protocol import NS_PUBSUB_CONFIG from nbxmpp.protocol import NS_DATA from nbxmpp.protocol import Node from nbxmpp.protocol import Iq @@ -27,6 +29,9 @@ from nbxmpp.protocol import isResultNode from nbxmpp.structs import StanzaHandler from nbxmpp.structs import PubSubEventData from nbxmpp.structs import CommonResult +from nbxmpp.structs import PubSubConfigResult +from nbxmpp.structs import PubSubPublishResult +from nbxmpp.modules.dataforms import extend_form from nbxmpp.util import call_on_response from nbxmpp.util import callback from nbxmpp.util import raise_error @@ -56,28 +61,36 @@ class PubSub: node, empty=True, deleted=True) return - retract = event.getTag('retract') - if retract is not None: - node = retract.getAttr('node') - item = retract.getTag('item') - id_ = item.getAttr('id') - properties.pubsub_event = PubSubEventData( - node, id_, item, retracted=True) + purge = event.getTag('purge') + if purge is not None: + node = purge.getAttr('node') + item = purge.getTag('item') + properties.pubsub_event = PubSubEventData(node, purged=True) return items = event.getTag('items') if items is not None: + node = items.getAttr('node') + + retract = items.getTag('retract') + if retract is not None: + id_ = retract.getAttr('id') + properties.pubsub_event = PubSubEventData( + node, id_, retracted=True) + return + if len(items.getChildren()) != 1: log.warning('PubSub event with more than one item') log.warning(stanza) - node = items.getAttr('node') + return + item = items.getTag('item') if item is None: return id_ = item.getAttr('id') properties.pubsub_event = PubSubEventData(node, id_, item) - @call_on_response('_default_response') + @call_on_response('_publish_result_received') def publish(self, jid, node, item, id_=None, options=None): query = Iq('set', to=jid) pubsub = query.addChild('pubsub', namespace=NS_PUBSUB) @@ -91,6 +104,87 @@ class PubSub: publish.addChild(node=options) return query + @callback + def _publish_result_received(self, stanza): + if not isResultNode(stanza): + return raise_error(log.warning, stanza) + + jid = stanza.getFrom() + pubsub = stanza.getTag('pubsub', namespace=NS_PUBSUB) + if pubsub is None: + return raise_error(log.warning, stanza, 'stanza-malformed') + + publish = pubsub.getTag('publish') + if publish is None: + return raise_error(log.warning, stanza, 'stanza-malformed') + + node = publish.getAttr('node') + item = publish.getTag('item') + if item is None: + return raise_error(log.warning, stanza, 'stanza-malformed') + + id_ = item.getAttr('id') + return PubSubPublishResult(jid, node, id_) + + @call_on_response('_default_response') + def retract(self, jid, node, id_, notify=True): + query = Iq('set', to=jid) + pubsub = query.addChild('pubsub', namespace=NS_PUBSUB) + attrs = {'node': node} + if notify: + attrs['notify'] = 'true' + retract = pubsub.addChild('retract', attrs=attrs) + retract.addChild('item', {'id': id_}) + return query + + @call_on_response('_default_response') + def set_node_configuration(self, jid, node, form): + log.info('Set configuration for %s %s', node, jid) + query = Iq('set', to=jid) + pubsub = query.addChild('pubsub', namespace=NS_PUBSUB_OWNER) + configure = pubsub.addChild('configure', {'node': node}) + form.setAttr('type', 'submit') + configure.addChild(node=form) + return query + + @call_on_response('_node_configuration_received') + def get_node_configuration(self, jid, node): + log.info('Request node configuration') + query = Iq('get', to=jid) + pubsub = query.addChild('pubsub', namespace=NS_PUBSUB_OWNER) + pubsub.addChild('configure', {'node': node}) + return query + + @callback + def _node_configuration_received(self, stanza): + if not isResultNode(stanza): + return raise_error(log.warning, stanza) + + jid = stanza.getFrom() + pubsub = stanza.getTag('pubsub', namespace=NS_PUBSUB_OWNER) + if pubsub is None: + return raise_error(log.warning, stanza, 'stanza-malformed', + 'No pubsub node found') + + configure = pubsub.getTag('configure') + if configure is None: + return raise_error(log.warning, stanza, 'stanza-malformed', + 'No configure node found') + + node = configure.getAttr('node') + + forms = configure.getTags('x', namespace=NS_DATA) + for form in forms: + dataform = extend_form(node=form) + form_type = dataform.vars.get('FORM_TYPE') + if form_type is None or form_type.value != NS_PUBSUB_CONFIG: + continue + log.info('Node configuration received from: %s', jid) + return PubSubConfigResult(jid=jid, node=node, form=dataform) + + return raise_error(log.warning, stanza, 'stanza-malformed', + 'No valid form type found') + @callback def _default_response(self, stanza): if not isResultNode(stanza): @@ -115,13 +209,23 @@ def get_pubsub_item(stanza): return items_node.getTag('item') -def get_bookmark_publish_options(): - # TODO: Make generic - options = Node(NS_DATA + ' x', - attrs={'type': 'submit'}) +def get_pubsub_items(stanza, node=None): + pubsub_node = stanza.getTag('pubsub') + items_node = pubsub_node.getTag('items') + if node is not None and items_node.getAttr('node') != node: + return + + if items_node is not None: + return items_node.getTags('item') + + +def get_publish_options(config): + options = Node(NS_DATA + ' x', attrs={'type': 'submit'}) field = options.addChild('field', attrs={'var': 'FORM_TYPE', 'type': 'hidden'}) field.setTagData('value', NS_PUBSUB_PUBLISH_OPTIONS) - field = options.addChild('field', attrs={'var': 'pubsub#access_model'}) - field.setTagData('value', 'whitelist') + + for var, value in config.items(): + field = options.addChild('field', attrs={'var': var}) + field.setTagData('value', value) return options ===================================== nbxmpp/protocol.py ===================================== @@ -51,6 +51,8 @@ NS_BIND = 'urn:ietf:params:xml:ns:xmpp-bind' NS_BLOCKING = 'urn:xmpp:blocking' # XEP-0191 NS_BOB = 'urn:xmpp:bob' # XEP-0231 NS_BOOKMARKS = 'storage:bookmarks' # XEP-0048 +NS_BOOKMARKS_2 = 'urn:xmpp:bookmarks:0' # XEP-0402 +NS_BOOKMARKS_COMPAT = 'urn:xmpp:bookmarks:0#compat' # XEP-0402 NS_BROWSE = 'jabber:iq:browse' NS_BROWSING = 'http://jabber.org/protocol/browsing' # XEP-0195 NS_BYTESTREAM = 'http://jabber.org/protocol/bytestreams' # XEP-0065 @@ -136,6 +138,7 @@ NS_PUBSUB = 'http://jabber.org/protocol/pubsub' # XEP-0060 NS_PUBSUB_EVENT = 'http://jabber.org/protocol/pubsub#event' NS_PUBSUB_PUBLISH_OPTIONS = NS_PUBSUB + '#publish-options' # XEP-0060 NS_PUBSUB_OWNER = 'http://jabber.org/protocol/pubsub#owner' # XEP-0060 +NS_PUBSUB_CONFIG = 'http://jabber.org/protocol/pubsub#node_config' # XEP-0060 NS_REGISTER = 'jabber:iq:register' NS_ROSTER = 'jabber:iq:roster' NS_ROSTERNOTES = 'storage:rosternotes' ===================================== nbxmpp/structs.py ===================================== @@ -70,8 +70,12 @@ HTTPAuthData.__new__.__defaults__ = (None, None, None, None) StanzaIDData = namedtuple('StanzaIDData', 'id by') StanzaIDData.__new__.__defaults__ = (None, None) -PubSubEventData = namedtuple('PubSubEventData', 'node id item data empty deleted retracted') -PubSubEventData.__new__.__defaults__ = (None, None, None, False, False, False) +PubSubEventData = namedtuple('PubSubEventData', 'node id item data empty deleted retracted purged') +PubSubEventData.__new__.__defaults__ = (None, None, None, False, False, False, False) + +PubSubConfigResult = namedtuple('PubSubConfigResult', 'jid node form') + +PubSubPublishResult = namedtuple('PubSubPublishResult', 'jid node id') MoodData = namedtuple('MoodData', 'mood text') View it on GitLab: https://dev.gajim.org/gajim/python-nbxmpp/commit/550decfd858f91fa5ada54ea04… -- View it on GitLab: https://dev.gajim.org/gajim/python-nbxmpp/commit/550decfd858f91fa5ada54ea04… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Fix show_unsubscribed_dialog
by Philipp Hörist 28 Sep '19

28 Sep '19

Philipp Hörist pushed to branch master at gajim / gajim Commits: 3de999d9 by Daniel Brötzmann at 2019-09-25T16:32:05Z Fix show_unsubscribed_dialog - - - - - 1 changed file: - gajim/gui_interface.py Changes: ===================================== gajim/gui_interface.py ===================================== @@ -405,9 +405,8 @@ class Interface: % obj.jid) def show_unsubscribed_dialog(self, account, contact): - def _remove(list_): - self.roster.on_req_usub(None, list_) - list_ = [(contact, account)] + def _remove(): + self.roster.on_req_usub(None, [(contact, account)]) NewConfirmationDialog( _('Subscription Removed'), @@ -417,7 +416,7 @@ class Interface: [DialogButton.make('Cancel', text=_('_No')), DialogButton.make('Remove', - callback=_remove(list_))]).show() + callback=_remove)]).show() # FIXME: Per RFC 3921, we can "deny" ack as well, but the GUI does # not show deny View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/3de999d93b9203304496f709b531eb3af0… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/3de999d93b9203304496f709b531eb3af0… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Presence: Dont show SubscriptionRequestWindow on auto approve
by Philipp Hörist 24 Sep '19

24 Sep '19

Philipp Hörist pushed to branch master at gajim / gajim Commits: 3647524a by Philipp Hörist at 2019-09-24T16:31:58Z Presence: Dont show SubscriptionRequestWindow on auto approve - - - - - 1 changed file: - gajim/common/modules/presence.py Changes: ===================================== gajim/common/modules/presence.py ===================================== @@ -250,6 +250,8 @@ class Presence(BaseModule): if auto_auth or is_transport or jid in self.jids_for_auto_auth: self.send_presence(fjid, 'subscribed') + self._log.info('Auto respond with subscribed: %s', jid) + return status = (properties.status or _('I would like to add you to my roster.')) View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/3647524ad328cdd971d1c286d8dbb0b56d… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/3647524ad328cdd971d1c286d8dbb0b56d… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] CI: Disable pylint E1133 for now
by Philipp Hörist 23 Sep '19

23 Sep '19

Philipp Hörist pushed to branch master at gajim / gajim Commits: 40ea0c9d by Philipp Hörist at 2019-09-23T20:34:03Z CI: Disable pylint E1133 for now see https://gitlab.gnome.org/GNOME/pygobject/issues/361 - - - - - 1 changed file: - scripts/dev/pylint-ci.sh Changes: ===================================== scripts/dev/pylint-ci.sh ===================================== @@ -2,7 +2,7 @@ PYLINT=${PYLINT:-pylint} "$PYLINT" --version -"$PYLINT" --disable=C0103,C0302,C0301,C0330,E0401,E0611,E1101,E1136,R0201,R0901,R0904,R0912,R0913,R0914,R0915,R0916,R1702,R1710,W0201,W0212,W0221,W0223,W0231,W0233,W0603,W0613 "$@" +"$PYLINT" --disable=C0103,C0302,C0301,C0330,E0401,E0611,E1101,E1133,E1136,R0201,R0901,R0904,R0912,R0913,R0914,R0915,R0916,R1702,R1710,W0201,W0212,W0221,W0223,W0231,W0233,W0603,W0613 "$@" # C0103 invalid-name # C0301 line-too-long @@ -11,6 +11,7 @@ PYLINT=${PYLINT:-pylint} # E0401 import-error # E0611 no-name-in-module # E1101 no-member +# E1133 not-an-iterable # E1136 unsubscriptable-object # R0201 no-self-use # R0901 too-many-ancestors View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/40ea0c9dae3bd0c0213c744c64eb6c357d… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/40ea0c9dae3bd0c0213c744c64eb6c357d… You're receiving this email because of your account on dev.gajim.org.

1 0

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

Commits September 2019