Commits October 2021

commits@gajim.org

1 participants
72 discussions

[Git][gajim/gajim][mainwindow] 3 commits: Main: Use first workspace for adding chat if nothing selected
by Daniel Brötzmann (＠wurstsalat) 25 Oct '21

25 Oct '21

Daniel Brötzmann pushed to branch mainwindow at gajim / gajim Commits: 206c9c2a by wurstsalat at 2021-10-26T00:22:39+02:00 Main: Use first workspace for adding chat if nothing selected - - - - - 6c9ef27c by wurstsalat at 2021-10-26T00:23:53+02:00 JumpToEndButton: Don't show_all when restoring window - - - - - bea23cc5 by wurstsalat at 2021-10-26T00:24:34+02:00 GroupchatState: Don't show_all when restoring window - - - - - 3 changed files: - gajim/gtk/conversation/jump_to_end_button.py - gajim/gtk/groupchat_state.py - gajim/gtk/main.py Changes: ===================================== gajim/gtk/conversation/jump_to_end_button.py ===================================== @@ -57,13 +57,14 @@ def __init__(self): self.show_all() self._count = 0 + self.set_no_show_all(True) def _on_jump_clicked(self, _button): self.emit('clicked') def toggle(self, visible): if visible: - self.show_all() + self.show() else: self.hide() self.reset_unread_count() ===================================== gajim/gtk/groupchat_state.py ===================================== @@ -48,6 +48,7 @@ def set_joining(self): def set_joined(self): self.hide() + self.set_no_show_all(True) def set_not_joined(self): self._ui.groupchat_state.set_visible_child_name('not-joining') ===================================== gajim/gtk/main.py ===================================== @@ -396,6 +396,8 @@ def _add_chat(self, _action, param): def add_chat(self, account, jid, type_, select=False): workspace_id = self.get_active_workspace() + if workspace_id is None: + workspace_id = self._workspace_side_bar.get_first_workspace() self._chat_page.add_chat_for_workspace(workspace_id, account, jid, View it on GitLab: https://dev.gajim.org/gajim/gajim/-/compare/26544cfb253a4213d817d233c8d26cb… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/-/compare/26544cfb253a4213d817d233c8d26cb… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/python-nbxmpp][master] Don’t use dataclasses.asdict()
by Philipp Hörist (＠lovetox) 25 Oct '21

25 Oct '21

Philipp Hörist pushed to branch master at gajim / python-nbxmpp Commits: 6f8890d7 by lovetox at 2021-10-25T23:54:24+02:00 Don’t use dataclasses.asdict() asdict() recurses into other objects, we don’t want this here - - - - - 1 changed file: - nbxmpp/structs.py Changes: ===================================== nbxmpp/structs.py ===================================== @@ -197,7 +197,11 @@ class RosterItem: groups=groups) def asdict(self): - return dataclasses.asdict(self) + return {'jid': self.jid, + 'name': self.name, + 'ask': self.ask, + 'subscription': self.subscription, + 'groups': self.groups} class DiscoInfo(namedtuple('DiscoInfo', 'stanza identities features dataforms timestamp')): View it on GitLab: https://dev.gajim.org/gajim/python-nbxmpp/-/commit/6f8890d74968119c12969dd8… -- View it on GitLab: https://dev.gajim.org/gajim/python-nbxmpp/-/commit/6f8890d74968119c12969dd8… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][mainwindow] AddContact: Always show Add button
by Daniel Brötzmann (＠wurstsalat) 24 Oct '21

24 Oct '21

Daniel Brötzmann pushed to branch mainwindow at gajim / gajim Commits: 26544cfb by wurstsalat at 2021-10-24T23:39:53+02:00 AddContact: Always show Add button - - - - - 1 changed file: - gajim/gtk/add_contact.py Changes: ===================================== gajim/gtk/add_contact.py ===================================== @@ -323,8 +323,6 @@ def __init__(self): self.show_all() def get_visible_buttons(self): - if self._contact.is_in_roster: - return ['back'] return ['back', 'add'] def get_default_button(self): View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/26544cfb253a4213d817d233c8d26cb7… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/26544cfb253a4213d817d233c8d26cb7… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][mainwindow] Groupchat: Implement click avatar to mention
by Daniel Brötzmann (＠wurstsalat) 24 Oct '21

24 Oct '21

Daniel Brötzmann pushed to branch mainwindow at gajim / gajim Commits: a856fc19 by wurstsalat at 2021-10-24T22:52:01+02:00 Groupchat: Implement click avatar to mention - - - - - 3 changed files: - gajim/gtk/controls/base.py - gajim/gtk/conversation/rows/message.py - gajim/gtk/conversation/view.py Changes: ===================================== gajim/gtk/controls/base.py ===================================== @@ -158,6 +158,7 @@ def __init__(self, widget_name, account, jid): # Create ConversationView and connect signals self.conversation_view = ConversationView(self.account, self.contact) self.conversation_view.connect('quote', self.on_quote) + self.conversation_view.connect('mention', self.on_mention) id_ = self.conversation_view.connect( 'key-press-event', self._on_conversation_view_key_press) @@ -717,6 +718,12 @@ def paste_clipboard_as_quote(self, _item: Gtk.MenuItem) -> None: def on_quote(self, _widget, text): self.insert_as_quote(text) + def on_mention(self, _widget, name): + gc_refer_to_nick_char = app.settings.get('gc_refer_to_nick_char') + text = f'{name}{gc_refer_to_nick_char} ' + message_buffer = self.msg_textview.get_buffer() + message_buffer.insert_at_cursor(text) + def _on_message_textview_paste_event(self, _texview): clipboard = Gtk.Clipboard.get(Gdk.SELECTION_CLIPBOARD) image = clipboard.wait_for_image() ===================================== gajim/gtk/conversation/rows/message.py ===================================== @@ -36,6 +36,7 @@ from ..message_widget import MessageWidget from ...preview import PreviewWidget from ...util import format_fingerprint +from ...util import get_cursor MERGE_TIMEFRAME = timedelta(seconds=120) @@ -141,7 +142,14 @@ def __init__(self, self._meta_box.pack_end(self._message_icons, False, True, 0) avatar = self._get_avatar(kind, name) self._avatar_image = Gtk.Image.new_from_surface(avatar) - avatar_placeholder = Gtk.Box() + + if self._is_groupchat: + avatar_placeholder = Gtk.EventBox() + avatar_placeholder.connect( + 'button-press-event', self._on_avatar_clicked, name) + avatar_placeholder.connect('realize', self._on_realize) + else: + avatar_placeholder = Gtk.Box() avatar_placeholder.set_size_request(AvatarSize.ROSTER, -1) avatar_placeholder.set_valign(Gtk.Align.START) avatar_placeholder.add(self._avatar_image) @@ -186,6 +194,14 @@ def _get_avatar(self, kind, name): return contact.get_avatar(AvatarSize.ROSTER, scale, add_show=False) + def _on_avatar_clicked(self, _widget, event, name): + if event.type == Gdk.EventType.BUTTON_PRESS and event.button == 1: + self.get_parent().on_mention(name) + + @staticmethod + def _on_realize(event_box): + event_box.get_window().set_cursor(get_cursor('pointer')) + def is_same_sender(self, message): return message.name == self.name ===================================== gajim/gtk/conversation/view.py ===================================== @@ -49,6 +49,11 @@ class ConversationView(Gtk.ListBox): None, (str, ) ), + 'mention': ( + GObject.SignalFlags.RUN_LAST | GObject.SignalFlags.ACTION, + None, + (str, ) + ), 'accept-call': ( GObject.SignalFlags.RUN_LAST | GObject.SignalFlags.ACTION, None, @@ -442,6 +447,9 @@ def show_error(self, id_, error): def on_quote(self, text): self.emit('quote', text) + def on_mention(self, name): + self.emit('mention', name) + def accept_call(self, event): self.emit('accept-call', event) View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/a856fc1924f73495f0327f18b986da23… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/a856fc1924f73495f0327f18b986da23… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][mainwindow] 2 commits: Tooltips: Add gui extension point for contact resources
by Daniel Brötzmann (＠wurstsalat) 24 Oct '21

24 Oct '21

Daniel Brötzmann pushed to branch mainwindow at gajim / gajim Commits: fd3fb091 by wurstsalat at 2021-10-24T20:12:17+02:00 Tooltips: Add gui extension point for contact resources - - - - - 212115c1 by wurstsalat at 2021-10-24T20:21:47+02:00 GCTooltip: Remove show - - - - - 3 changed files: - gajim/data/gui/tooltip_gc_contact.ui → gajim/data/gui/groupchat_roster_tooltip.ui - gajim/data/gui/roster_tooltip.ui - gajim/gtk/tooltips.py Changes: ===================================== gajim/data/gui/tooltip_gc_contact.ui → gajim/data/gui/groupchat_roster_tooltip.ui ===================================== @@ -1,49 +1,53 @@ <?xml version="1.0" encoding="UTF-8"?> - + <interface> - <requires lib="gtk+" version="3.22"/> + <requires lib="gtk+" version="3.24"/> +  <object class="GtkGrid" id="tooltip_grid"> <property name="visible">True</property> - <property name="can_focus">False</property> - <property name="column_spacing">6</property> + <property name="can-focus">False</property> + <property name="column-spacing">6</property> <child> <object class="GtkImage" id="avatar"> <property name="visible">True</property> - <property name="can_focus">False</property> + <property name="can-focus">False</property> + <property name="valign">start</property> + <property name="icon-name">image-missing</property> + <property name="icon_size">6</property> </object> <packing> - <property name="left_attach">0</property> - <property name="top_attach">0</property> + <property name="left-attach">0</property> + <property name="top-attach">0</property> <property name="height">6</property> </packing> </child> <child> <object class="GtkLabel" id="jid"> <property name="visible">True</property> - <property name="can_focus">False</property> + <property name="can-focus">False</property> <property name="halign">start</property> <property name="hexpand">True</property> <property name="ellipsize">end</property> - <property name="single_line_mode">True</property> - <property name="max_width_chars">36</property> + <property name="single-line-mode">True</property> + <property name="max-width-chars">36</property> <attributes> <attribute name="weight" value="bold"/> </attributes> </object> <packing> - <property name="left_attach">1</property> - <property name="top_attach">1</property> + <property name="left-attach">1</property> + <property name="top-attach">1</property> </packing> </child> <child> <object class="GtkLabel" id="nick"> <property name="visible">True</property> - <property name="can_focus">False</property> + <property name="can-focus">False</property> <property name="halign">start</property> <property name="valign">start</property> <property name="ellipsize">end</property> - <property name="single_line_mode">True</property> - <property name="max_width_chars">25</property> + <property name="single-line-mode">True</property> + <property name="max-width-chars">25</property> <attributes> <attribute name="weight" value="bold"/> </attributes> @@ -52,44 +56,32 @@ </style> </object> <packing> - <property name="left_attach">1</property> - <property name="top_attach">0</property> - </packing> - </child> - <child> - <object class="GtkLabel" id="user_show"> - <property name="visible">True</property> - <property name="can_focus">False</property> - <property name="halign">start</property> - <property name="valign">start</property> - </object> - <packing> - <property name="left_attach">1</property> - <property name="top_attach">3</property> + <property name="left-attach">1</property> + <property name="top-attach">0</property> </packing> </child> <child> <object class="GtkLabel" id="fillelement"> <property name="visible">True</property> - <property name="can_focus">False</property> + <property name="can-focus">False</property> <property name="halign">start</property> <property name="vexpand">True</property> </object> <packing> - <property name="left_attach">1</property> - <property name="top_attach">5</property> + <property name="left-attach">1</property> + <property name="top-attach">5</property> </packing> </child> <child> <object class="GtkLabel" id="status"> <property name="visible">True</property> - <property name="can_focus">False</property> + <property name="can-focus">False</property> <property name="halign">start</property> <property name="valign">start</property> <property name="wrap">True</property> - <property name="wrap_mode">word-char</property> + <property name="wrap-mode">word-char</property> <property name="ellipsize">end</property> - <property name="max_width_chars">40</property> + <property name="max-width-chars">40</property> <property name="lines">3</property> <property name="xalign">0</property> <attributes> @@ -97,21 +89,42 @@ </attributes> </object> <packing> - <property name="left_attach">1</property> - <property name="top_attach">4</property> + <property name="left-attach">1</property> + <property name="top-attach">4</property> </packing> </child> <child> <object class="GtkLabel" id="affiliation"> <property name="visible">True</property> - <property name="can_focus">False</property> + <property name="can-focus">False</property> <property name="halign">start</property> <property name="valign">start</property> </object> <packing> - <property name="left_attach">1</property> - <property name="top_attach">2</property> + <property name="left-attach">1</property> + <property name="top-attach">2</property> </packing> </child> + <child> + <placeholder/> + </child> + <child> + <placeholder/> + </child> + <child> + <placeholder/> + </child> + <child> + <placeholder/> + </child> + <child> + <placeholder/> + </child> + <child> + <placeholder/> + </child> + <child> + <placeholder/> + </child> </object> </interface> ===================================== gajim/data/gui/roster_tooltip.ui ===================================== @@ -193,15 +193,12 @@ </packing> </child> <child> -  - <object class="GtkGrid" id="resource_grid"> + <object class="GtkBox" id="resources_box"> <property name="visible">True</property> <property name="can-focus">False</property> <property name="halign">start</property> <property name="valign">start</property> - <child> - <placeholder/> - </child> + <property name="orientation">vertical</property> <child> <placeholder/> </child> ===================================== gajim/gtk/tooltips.py ===================================== @@ -48,29 +48,28 @@ from .util import format_activity from .util import format_tune from .util import format_location -from .util import get_css_show_class log = logging.getLogger('gajim.gui.tooltips') class GCTooltip: def __init__(self): - self.contact = None + self._contact = None - self._ui = get_builder('tooltip_gc_contact.ui') + self._ui = get_builder('groupchat_roster_tooltip.ui') def clear_tooltip(self): - self.contact = None + self._contact = None def get_tooltip(self, contact): - if self.contact == contact: + if self._contact == contact: return True, self._ui.tooltip_grid self._populate_grid(contact) - self.contact = contact + self._contact = contact return False, self._ui.tooltip_grid - def _hide_grid_childs(self): + def _hide_grid_children(self): """ Hide all Elements of the Tooltip Grid """ @@ -81,7 +80,7 @@ def _populate_grid(self, contact): """ Populate the Tooltip Grid with data of from the contact """ - self._hide_grid_childs() + self._hide_grid_children() self._ui.nick.set_text(contact.name) self._ui.nick.show() @@ -93,12 +92,6 @@ def _populate_grid(self, contact): self._ui.status.set_text(status) self._ui.status.show() - # Status - show = helpers.get_uf_show(contact.show.value) - self._ui.user_show.set_text(show) - colorize_status(self._ui.user_show, contact.show.value) - self._ui.user_show.show() - # JID if contact.real_jid is not None: self._ui.jid.set_text(str(contact.real_jid.bare)) @@ -134,7 +127,7 @@ def __init__(self): def clear_tooltip(self): self._row = None - for widget in self._ui.resource_grid.get_children(): + for widget in self._ui.resources_box.get_children(): widget.destroy() for widget in self._ui.tooltip_grid.get_children(): widget.hide() @@ -165,8 +158,7 @@ def _populate_grid(self, contact): self._ui.jid.show() # Resources with show, status, priority - resources = contact.get_resources() - self._build_resource_grid(resources, scale) + self._add_resources(contact) # Subscription if contact.subscription and contact.subscription != 'both': @@ -180,7 +172,7 @@ def _populate_grid(self, contact): # self._append_pep_info(contact) app.plugin_manager.gui_extension_point( - 'roster_tooltip_populate', self, contact, self._ui.tooltip_grid) + 'roster_tooltip_populate', self, contact) # This sets the bottom-most widget to expand, in case the avatar # takes more space than the labels @@ -194,39 +186,42 @@ def _populate_grid(self, contact): break last_widget.set_vexpand(True) - def _build_resource_grid(self, resources, scale): - row_num = 0 - for contact in resources: + def _add_resources(self, contact): + for resource in contact.iter_resources(): + resource_box = Gtk.Box(orientation=Gtk.Orientation.VERTICAL) + show_surface = get_show_circle( - contact.show, AvatarSize.SHOW_CIRCLE, scale) + resource.show, + AvatarSize.SHOW_CIRCLE, + self._ui.tooltip_grid.get_scale_factor()) show_image = Gtk.Image.new_from_surface(show_surface) show_image.set_halign(Gtk.Align.START) show_image.set_valign(Gtk.Align.CENTER) - self._ui.resource_grid.attach(show_image, 0, row_num, 1, 1) - resource_string = GLib.markup_escape_text(contact.jid.resource) + resource_string = GLib.markup_escape_text(resource.jid.resource) resource_label = Gtk.Label() resource_label.set_halign(Gtk.Align.START) resource_label.set_xalign(0) resource_label.set_ellipsize(Pango.EllipsizeMode.END) resource_label.set_max_width_chars(30) - resource_label.set_text(f'{resource_string} ({contact.priority})') - self._ui.resource_grid.attach(resource_label, 1, row_num, 1, 1) + resource_label.set_text(f'{resource_string} ({resource.priority})') + + base_box = Gtk.Box(spacing=6) + base_box.add(show_image) + base_box.add(resource_label) + resource_box.add(base_box) - if contact.status: - row_num += 1 - status_text = GLib.markup_escape_text(contact.status) + if resource.status: + status_text = GLib.markup_escape_text(resource.status) status_label = Gtk.Label(label=status_text) status_label.set_halign(Gtk.Align.START) status_label.set_xalign(0) status_label.set_ellipsize(Pango.EllipsizeMode.END) status_label.set_max_width_chars(30) - self._ui.resource_grid.attach( - status_label, 1, row_num, 1, 1) + resource_box.add(status_label) - if contact.idle_time: - row_num += 1 - idle_time = time.localtime(contact.idle_time) + if resource.idle_time: + idle_time = time.localtime(resource.idle_time) idle_time = datetime(*(idle_time[:6])) current = datetime.now() if idle_time.date() == current.date(): @@ -237,12 +232,16 @@ def _build_resource_grid(self, resources, scale): idle_label = Gtk.Label(label=idle_text) idle_label.set_halign(Gtk.Align.START) idle_label.set_xalign(0) - self._ui.resource_grid.attach( - idle_label, 1, row_num, 1, 1) + resource_box.add(idle_label) - row_num += 1 + app.plugin_manager.gui_extension_point( + 'roster_tooltip_resource_populate', + resource_box, + resource) - self._ui.resource_grid.show_all() + self._ui.resources_box.add(resource_box) + + self._ui.resources_box.show_all() def _append_pep_info(self, contact): if PEPEventType.MOOD in contact.pep: @@ -367,16 +366,3 @@ def _get_current_status(file_props): return Q_('?transfer status:Transferring') return Q_('?transfer status:Not started') return Q_('?transfer status:Not started') - - -def colorize_status(widget, show): - """ - Colorize the status message inside the tooltip by it's semantics. - """ - css_class = get_css_show_class(show)[14:] - style = widget.get_style_context() - # Remove all css classes with gajim-status- prefix - for css_cls in style.list_classes(): - if css_cls.startswith('gajim-status-'): - style.remove_class(css_cls) - style.add_class(f'gajim-status-{css_class}') View it on GitLab: https://dev.gajim.org/gajim/gajim/-/compare/7289a04e41aba8d1464371137bade0a… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/-/compare/7289a04e41aba8d1464371137bade0a… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][mainwindow] 3 commits: Main: Don’t open control for chatstates etc.
by Daniel Brötzmann (＠wurstsalat) 24 Oct '21

24 Oct '21

Daniel Brötzmann pushed to branch mainwindow at gajim / gajim Commits: 4a8c9e89 by wurstsalat at 2021-10-24T15:59:08+02:00 Main: Don’t open control for chatstates etc. - - - - - ce0bf6a2 by wurstsalat at 2021-10-24T16:05:15+02:00 NotificationManager: Fix tray notifications - - - - - 7289a04e by wurstsalat at 2021-10-24T16:08:17+02:00 Notifications: Fix getting icon - - - - - 3 changed files: - gajim/gtk/main.py - gajim/gtk/notification.py - gajim/gtk/notification_manager.py Changes: ===================================== gajim/gtk/main.py ===================================== @@ -560,6 +560,9 @@ def _on_event(self, event): if not self.chat_exists(event.account, event.jid): if event.name == 'message-received': + if not event.properties.body: + # Don’t open control on chatstate etc. + return if event.properties.is_muc_pm: self.add_private_chat(event.account, event.properties.jid, ===================================== gajim/gtk/notification.py ===================================== @@ -133,7 +133,7 @@ def _on_notification(self, event): icon_name = event.icon_name else: icon_name = NOTIFICATION_ICONS.get( - event.notif_detail, 'mail-message-new') + notif_detail, 'mail-message-new') self._issue_notification( event.notif_type, ===================================== gajim/gtk/notification_manager.py ===================================== @@ -143,8 +143,8 @@ def add_subscription_request(self, event): app.nec.push_incoming_event( NetworkEvent('notification', - account=self.account, - jid=self.contact.jid, + account=self._account, + jid=event.jid, notif_type='subscription-request', title=_('Subscription Request'), text=text)) @@ -162,8 +162,8 @@ def add_unsubscribed(self, event): app.nec.push_incoming_event( NetworkEvent('notification', - account=self.account, - jid=self.contact.jid, + account=self._account, + jid=event.jid, notif_type='unsubscribed', title=_('Contact Unsubscribed'), text=text)) @@ -186,8 +186,8 @@ def add_invitation_received(self, event): app.nec.push_incoming_event( NetworkEvent('notification', - account=self.account, - jid=self.contact.jid, + account=self._account, + jid=jid, notif_type='group-chat-invitation', title=_('Group Chat Invitation'), text=text)) View it on GitLab: https://dev.gajim.org/gajim/gajim/-/compare/638e5bf31d1bfbb2fee9453524780c9… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/-/compare/638e5bf31d1bfbb2fee9453524780c9… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/python-nbxmpp][nbxmpp_2.0] Namespaces: Fix bookmarks compat namespace
by Philipp Hörist (＠lovetox) 23 Oct '21

23 Oct '21

Philipp Hörist pushed to branch nbxmpp_2.0 at gajim / python-nbxmpp Commits: 8fffacf1 by lovetox at 2021-10-23T22:52:06+02:00 Namespaces: Fix bookmarks compat namespace - - - - - 1 changed file: - nbxmpp/namespaces.py Changes: ===================================== nbxmpp/namespaces.py ===================================== @@ -35,7 +35,7 @@ class _Namespaces: BOB: str = 'urn:xmpp:bob' BOOKMARKS: str = 'storage:bookmarks' BOOKMARKS_1: str = 'urn:xmpp:bookmarks:1' - BOOKMARKS_COMPAT: str = 'urn:xmpp:bookmarks:0#compat' + BOOKMARKS_COMPAT: str = 'urn:xmpp:bookmarks:1#compat' BOOKMARKS_COMPAT_PEP: str = 'urn:xmpp:bookmarks:1#compat-pep' BOOKMARK_CONVERSION: str = 'urn:xmpp:bookmarks-conversion:0' BROWSE: str = 'jabber:iq:browse' View it on GitLab: https://dev.gajim.org/gajim/python-nbxmpp/-/commit/8fffacf11567f2641e82aebd… -- View it on GitLab: https://dev.gajim.org/gajim/python-nbxmpp/-/commit/8fffacf11567f2641e82aebd… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/python-nbxmpp][master] 9 commits: Remove unused dict
by Philipp Hörist (＠lovetox) 23 Oct '21

23 Oct '21

Philipp Hörist pushed to branch master at gajim / python-nbxmpp Commits: bc2870f8 by lovetox at 2021-10-16T17:22:26+02:00 Remove unused dict - - - - - f2a4cdf0 by lovetox at 2021-10-16T17:23:01+02:00 Use generate_id() instead of get_stanza_id() - - - - - cd079530 by lovetox at 2021-10-16T17:23:35+02:00 Add annotations - - - - - b2a46b62 by lovetox at 2021-10-16T17:23:49+02:00 Remove unused method - - - - - b48f0f6c by lovetox at 2021-10-17T11:57:33+02:00 Add annotations - - - - - f5a9acc0 by lovetox at 2021-10-17T12:15:03+02:00 Refactor b64decode() - - - - - 53bfd31b by lovetox at 2021-10-17T12:26:31+02:00 Refactor b64encode() - - - - - bc7cac2b by lovetox at 2021-10-17T14:05:20+02:00 Add annotations - - - - - e3d66d9c by lovetox at 2021-10-23T22:50:23+02:00 Namespaces: Fix bookmarks compat namespace - - - - - 13 changed files: - nbxmpp/auth.py - nbxmpp/client.py - nbxmpp/modules/bits_of_binary.py - nbxmpp/modules/ibb.py - nbxmpp/modules/omemo.py - nbxmpp/modules/openpgp.py - nbxmpp/modules/user_avatar.py - nbxmpp/modules/vcard_temp.py - nbxmpp/namespaces.py - nbxmpp/protocol.py - nbxmpp/simplexml.py - nbxmpp/structs.py - nbxmpp/util.py Changes: ===================================== nbxmpp/auth.py ===================================== @@ -288,7 +288,7 @@ class GSSAPI: self._client.send_nonza(node) def response(self, server_message, *args, **kwargs): - server_message = b64decode(server_message, bytes) + server_message = b64decode(server_message) try: if not self.ctx.complete: output_token = self.ctx.step(server_message) @@ -350,14 +350,14 @@ class SCRAM: self._client.send_nonza(node) def response(self, server_first_message): - server_first_message = b64decode(server_first_message) + server_first_message = b64decode(server_first_message).decode() challenge = self._scram_parse(server_first_message) client_nonce = challenge['r'][:self.nonce_length] if client_nonce != self._client_nonce: raise AuthFail('Invalid client nonce received from server') - salt = b64decode(challenge['s'], bytes) + salt = b64decode(challenge['s']) iteration_count = int(challenge['i']) if iteration_count < 4096: @@ -397,9 +397,9 @@ class SCRAM: self._client.send_nonza(node) def success(self, server_last_message): - server_last_message = b64decode(server_last_message) + server_last_message = b64decode(server_last_message).decode() success = self._scram_parse(server_last_message) - server_signature = b64decode(success['v'], bytes) + server_signature = b64decode(success['v']) if server_signature != self._server_signature: raise AuthFail('Invalid server signature') ===================================== nbxmpp/client.py ===================================== @@ -45,7 +45,7 @@ from nbxmpp.const import ConnectionProtocol from nbxmpp.const import Mode from nbxmpp.dispatcher import StanzaDispatcher from nbxmpp.util import get_stream_header -from nbxmpp.util import get_stanza_id +from nbxmpp.util import generate_id from nbxmpp.util import Observable from nbxmpp.util import validate_stream_header from nbxmpp.util import LogAdapter @@ -576,7 +576,7 @@ class Client(Observable): id_ = stanza.getID() if id_ is None: - id_ = get_stanza_id() + id_ = generate_id() stanza.setID(id_) if callback is not None: ===================================== nbxmpp/modules/bits_of_binary.py ===================================== @@ -15,9 +15,12 @@ # You should have received a copy of the GNU General Public License # along with this program; If not, see <http://www.gnu.org/licenses/>. +from typing import Optional + import logging import hashlib +from nbxmpp.simplexml import Node from nbxmpp.namespaces import Namespace from nbxmpp.structs import BobData from nbxmpp.util import b64decode @@ -25,7 +28,7 @@ from nbxmpp.util import b64decode log = logging.getLogger('nbxmpp.m.bob') -def parse_bob_data(stanza): +def parse_bob_data(stanza: Node) -> Optional[BobData]: data_node = stanza.getTag('data', namespace=Namespace.BOB) if data_node is None: return None @@ -57,7 +60,7 @@ def parse_bob_data(stanza): return None try: - bob_data = b64decode(bob_data, return_type=bytes) + bob_data = b64decode(bob_data) except Exception: log.warning('Unable to decode data') log.exception(stanza) ===================================== nbxmpp/modules/ibb.py ===================================== @@ -113,7 +113,7 @@ class IBB(BaseModule): raise NodeProcessed try: - decoded_data = b64decode(data.getData(), return_type=bytes) + decoded_data = b64decode(data.getData()) except Exception: self._log.exception('Failed to decode IBB data') self._client.send_stanza(ErrorStanza(stanza, ERR_BAD_REQUEST)) ===================================== nbxmpp/modules/omemo.py ===================================== @@ -194,7 +194,7 @@ def _parse_omemo_message(stanza): iv_node = header.getTag('iv') try: - iv = b64decode(iv_node.getData(), bytes) + iv = b64decode(iv_node.getData()) except Exception as error: raise MalformedStanzaError('failed to decode iv: %s' % error, stanza) @@ -202,7 +202,7 @@ def _parse_omemo_message(stanza): payload_node = encrypted.getTag('payload') if payload_node is not None: try: - payload = b64decode(payload_node.getData(), bytes) + payload = b64decode(payload_node.getData()) except Exception as error: raise MalformedStanzaError('failed to decode payload: %s' % error, stanza) @@ -227,7 +227,7 @@ def _parse_omemo_message(stanza): raise MalformedStanzaError(error, stanza) try: - keys[int(rid)] = (b64decode(kn.getData(), bytes), prekey) + keys[int(rid)] = (b64decode(kn.getData()), prekey) except Exception as error: raise MalformedStanzaError('failed to decode key: %s' % error, stanza) @@ -273,8 +273,7 @@ def _parse_bundle(item): result = {} signed_prekey_node = bundle.getTag('signedPreKeyPublic') try: - result['spk'] = {'key': b64decode(signed_prekey_node.getData(), - bytes)} + result['spk'] = {'key': b64decode(signed_prekey_node.getData())} except Exception as error: error = 'Failed to decode signedPreKeyPublic: %s' % error raise MalformedStanzaError(error, item) @@ -287,15 +286,14 @@ def _parse_bundle(item): signed_signature_node = bundle.getTag('signedPreKeySignature') try: - result['spk_signature'] = b64decode(signed_signature_node.getData(), - bytes) + result['spk_signature'] = b64decode(signed_signature_node.getData()) except Exception as error: error = 'Failed to decode signedPreKeySignature: %s' % error raise MalformedStanzaError(error, item) identity_key_node = bundle.getTag('identityKey') try: - result['ik'] = b64decode(identity_key_node.getData(), bytes) + result['ik'] = b64decode(identity_key_node.getData()) except Exception as error: error = 'Failed to decode IdentityKey: %s' % error raise MalformedStanzaError(error, item) @@ -312,7 +310,7 @@ def _parse_bundle(item): raise MalformedStanzaError('Invalid prekey: %s' % error, item) try: - key = b64decode(prekey.getData(), bytes) + key = b64decode(prekey.getData()) except Exception as error: raise MalformedStanzaError( 'Failed to decode preKeyPublic: %s' % error, item) ===================================== nbxmpp/modules/openpgp.py ===================================== @@ -74,7 +74,7 @@ class OpenPGP(BaseModule): self._log.info('Encrypted message received') try: - properties.openpgp = b64decode(data, return_type=bytes) + properties.openpgp = b64decode(data) except Exception: self._log.warning('b64decode failed') self._log.warning(stanza) @@ -390,7 +390,7 @@ def _parse_public_key(jid, item): raise ValueError('data node missing') try: - key = b64decode(data.getData(), return_type=bytes) + key = b64decode(data.getData()) except Exception as error: raise ValueError(f'decoding error: {error}') @@ -432,7 +432,7 @@ def _parse_secret_key(item): raise ValueError('secretkey data missing') try: - key = b64decode(data, return_type=bytes) + key = b64decode(data) except Exception as error: raise ValueError(f'decoding error: {error}') ===================================== nbxmpp/modules/user_avatar.py ===================================== @@ -220,7 +220,7 @@ def _get_avatar_data(item, id_): raise MalformedStanzaError('data node empty', item) try: - avatar = b64decode(data, return_type=bytes) + avatar = b64decode(data) except Exception as error: raise MalformedStanzaError(f'decoding error: {error}', item) ===================================== nbxmpp/modules/vcard_temp.py ===================================== @@ -138,6 +138,6 @@ class VCard: if not avatar: return None, None - avatar = b64decode(avatar, return_type=bytes) + avatar = b64decode(avatar) avatar_sha = hashlib.sha1(avatar).hexdigest() return avatar, avatar_sha ===================================== nbxmpp/namespaces.py ===================================== @@ -35,7 +35,7 @@ class _Namespaces: BOB: str = 'urn:xmpp:bob' BOOKMARKS: str = 'storage:bookmarks' BOOKMARKS_1: str = 'urn:xmpp:bookmarks:1' - BOOKMARKS_COMPAT: str = 'urn:xmpp:bookmarks:0#compat' + BOOKMARKS_COMPAT: str = 'urn:xmpp:bookmarks:1#compat' BOOKMARKS_COMPAT_PEP: str = 'urn:xmpp:bookmarks:1#compat-pep' BOOKMARK_CONVERSION: str = 'urn:xmpp:bookmarks-conversion:0' BROWSE: str = 'jabber:iq:browse' ===================================== nbxmpp/protocol.py ===================================== @@ -18,6 +18,13 @@ data structures, including jabber-objects like JID or different stanzas and sub- stanzas) handling routines """ +from __future__ import annotations + +from typing import Any +from typing import Union +from typing import Optional +from typing import cast + import time import hashlib import functools @@ -241,27 +248,6 @@ _errorcodes = { 'cancel': 'invalid-from' } -_status_conditions = { - 'realjid-public': 100, - 'affiliation-changed': 101, - 'unavailable-shown': 102, - 'unavailable-not-shown': 103, - 'configuration-changed': 104, - 'self-presence': 110, - 'logging-enabled': 170, - 'logging-disabled': 171, - 'non-anonymous': 172, - 'semi-anonymous': 173, - 'fully-anonymous': 174, - 'room-created': 201, - 'nick-assigned': 210, - 'banned': 301, - 'new-nick': 303, - 'kicked': 307, - 'removed-affiliation': 321, - 'removed-membership': 322, - 'removed-shutdown': 332, -} _localpart_disallowed_chars = set('"&\'/:<>@') _localpart_escape_chars = ' "&\'/:<>@' @@ -490,7 +476,7 @@ def deprecation_warning(message): @functools.lru_cache(maxsize=None) -def validate_localpart(localpart): +def validate_localpart(localpart: str) -> str: if not localpart or len(localpart.encode()) > 1023: raise LocalpartByteLimit @@ -505,7 +491,7 @@ def validate_localpart(localpart): @functools.lru_cache(maxsize=None) -def validate_resourcepart(resourcepart): +def validate_resourcepart(resourcepart: str) -> str: if not resourcepart or len(resourcepart.encode()) > 1023: raise ResourcepartByteLimit @@ -517,7 +503,7 @@ def validate_resourcepart(resourcepart): @functools.lru_cache(maxsize=None) -def validate_domainpart(domainpart): +def validate_domainpart(domainpart: str) -> str: if not domainpart: raise DomainpartByteLimit @@ -541,12 +527,12 @@ def validate_domainpart(domainpart): @functools.lru_cache(maxsize=None) -def idna_encode(domain): +def idna_encode(domain: str) -> str: return idna.encode(domain, uts46=True).decode() @functools.lru_cache(maxsize=None) -def escape_localpart(localpart): +def escape_localpart(localpart: str) -> str: # https://xmpp.org/extensions/xep-0106.html#bizrules-algorithm # # If there are any instances of character sequences that correspond @@ -568,7 +554,7 @@ def escape_localpart(localpart): @functools.lru_cache(maxsize=None) -def unescape_localpart(localpart): +def unescape_localpart(localpart: str) -> str: if localpart.startswith('\\20') or localpart.endswith('\\20'): # Escaped JIDs are not allowed to start or end with \20 # so this localpart must be already unescaped @@ -587,11 +573,15 @@ def unescape_localpart(localpart): @dataclass(frozen=True) class JID: - localpart: str = None - domain: str = None - resource: str = None + localpart: Optional[str] = None + domain: Optional[str] = None + resource: Optional[str] = None + + def __init__(self, + localpart: Optional[str] = None, + domain: Optional[str] = None, + resource: Optional[str] = None): - def __init__(self, localpart=None, domain=None, resource=None): if localpart is not None: localpart = validate_localpart(localpart) object.__setattr__(self, "localpart", localpart) @@ -605,7 +595,7 @@ class JID: @classmethod @functools.lru_cache(maxsize=None) - def from_string(cls, jid_string): + def from_string(cls, jid_string: str) -> JID: # https://tools.ietf.org/html/rfc7622#section-3.2 # Remove any portion from the first '/' character to the end of the @@ -630,7 +620,7 @@ class JID: @classmethod @functools.lru_cache(maxsize=None) - def from_user_input(cls, user_input, escaped=False): + def from_user_input(cls, user_input: str, escaped: bool = False) -> JID: # Use this if we want JIDs to be escaped according to XEP-0106 # The standard JID parsing cannot be applied because user_input is # not a valid JID. @@ -661,11 +651,11 @@ class JID: domain=domainpart, resource=None) - def __str__(self): + def __str__(self) -> str: if self.localpart: jid = f'{self.localpart}(a){self.domain}' else: - jid = self.domain + jid = cast(str, self.domain) if self.resource is not None: return f'{jid}/{self.resource}' @@ -674,7 +664,7 @@ class JID: def __hash__(self): return hash(str(self)) - def __eq__(self, other): + def __eq__(self, other: Union[str, JID]) -> bool: if isinstance(other, str): deprecation_warning('comparing string with JID is deprected') try: @@ -682,57 +672,54 @@ class JID: except Exception: return False - if not isinstance(other, JID): - raise TypeError('eq with type (%s) not supported' % type(other)) - return (self.localpart == other.localpart and self.domain == other.domain and self.resource == other.resource) - def __ne__(self, other): + def __ne__(self, other: Union[str, JID]) -> bool: return not self.__eq__(other) - def domain_to_ascii(self): + def domain_to_ascii(self) -> str: return idna_encode(self.domain) @property - def bare(self): + def bare(self) -> Optional[str]: if self.localpart is not None: return f'{self.localpart}(a){self.domain}' return self.domain @property - def is_bare(self): + def is_bare(self) -> bool: return self.resource is None - def new_as_bare(self): + def new_as_bare(self) -> JID: if self.resource is None: return self new = asdict(self) new.pop('resource') return JID(**new) - def bare_match(self, other): + def bare_match(self, other: Union[str, JID]) -> bool: if isinstance(other, str): other = JID.from_string(other) return self.bare == other.bare @property - def is_domain(self): + def is_domain(self) -> bool: return self.localpart is None and self.resource is None @property - def is_full(self): + def is_full(self) -> bool: return (self.localpart is not None and self.domain is not None and self.resource is not None) - def new_with(self, **kwargs): + def new_with(self, **kwargs: dict[str, str]) -> JID: new = asdict(self) new.update(kwargs) return JID(**new) - def to_user_string(self, show_punycode=True): + def to_user_string(self, show_punycode: bool = True) -> str: domain = self.domain_to_ascii() if domain.startswith('xn--') and show_punycode: domain_encoded = f' ({domain})' @@ -748,7 +735,7 @@ class JID: return f'{localpart}(a){self.domain}{domain_encoded}' return f'{localpart}(a){self.domain}/{self.resource}{domain_encoded}' - def copy(self): + def copy(self) -> JID: deprecation_warning('copy() is not needed, JID is immutable') return self @@ -757,7 +744,7 @@ class StreamErrorNode(Node): def __init__(self, node): Node.__init__(self, node=node) - self._text = {} + self._text: dict[Optional[str], str] = {} text_elements = self.getTags('text', namespace=Namespace.XMPP_STREAMS) for element in text_elements: @@ -765,14 +752,14 @@ class StreamErrorNode(Node): text = element.getData() self._text[lang] = text - def get_condition(self): + def get_condition(self) -> Optional[str]: for tag in self.getChildren(): if (tag.getName() != 'text' and tag.getNamespace() == Namespace.XMPP_STREAMS): return tag.getName() return None - def get_text(self, pref_lang=None): + def get_text(self, pref_lang: Optional[str] = None) -> str: if pref_lang is not None: text = self._text.get(pref_lang) if text is not None: @@ -905,13 +892,13 @@ class Protocol(Node): return JID.from_string(attr) return attr - def getID(self): + def getID(self) -> Optional[str]: """ Return the value of the 'id' attribute """ return self.getAttr('id') - def setTo(self, val): + def setTo(self, val: Union[str, JID]): """ Set the value of the 'to' attribute """ @@ -919,13 +906,13 @@ class Protocol(Node): val = JID.from_string(val) self.setAttr('to', val) - def getType(self): + def getType(self) -> Optional[str]: """ Return the value of the 'type' attribute """ return self.getAttr('type') - def setFrom(self, val): + def setFrom(self, val: Union[str, JID]): """ Set the value of the 'from' attribute """ @@ -933,13 +920,13 @@ class Protocol(Node): val = JID.from_string(val) self.setAttr('from', val) - def setType(self, val): + def setType(self, val: str): """ Set the value of the 'type' attribute """ self.setAttr('type', val) - def setID(self, val): + def setID(self, val: str): """ Set the value of the 'id' attribute """ @@ -1056,7 +1043,11 @@ class Protocol(Node): props.append(prop) return props - def getTag(self, name, attrs=None, namespace=None, protocol=False): + def getTag(self, + name: str, + attrs: Optional[dict[str, Any]] = None, + namespace: Optional[str] = None, + protocol: bool = False) -> Optional[Node]: """ Return the Node instance for the tag. If protocol is True convert to a new Protocol/Message instance. @@ -1068,14 +1059,14 @@ class Protocol(Node): return Protocol(node=tag) return tag - def __setitem__(self, item, val): + def __setitem__(self, item: str, val: Union[str, JID]): """ Set the item 'item' to the value 'val' """ if item in ['to', 'from']: if not isinstance(val, JID): val = JID.from_string(val) - return self.setAttr(item, val) + self.setAttr(item, val) class Message(Protocol): @@ -1658,13 +1649,15 @@ class Hashes2(Node): class BindRequest(Iq): - def __init__(self, resource): - if resource is not None: - resource = Node('resource', payload=resource) + def __init__(self, resource: Optional[str]): + if resource is None: + res = resource + else: + res = Node('resource', payload=resource) Iq.__init__(self, typ='set') self.addChild(node=Node('bind', {'xmlns': Namespace.BIND}, - payload=resource)) + payload=res)) class TLSRequest(Node): @@ -1679,7 +1672,7 @@ class SessionRequest(Iq): class StreamHeader(Node): - def __init__(self, domain, lang=None): + def __init__(self, domain: str, lang: Optional[str] = None): if lang is None: lang = 'en' Node.__init__(self, @@ -1692,7 +1685,7 @@ class StreamHeader(Node): class WebsocketOpenHeader(Node): - def __init__(self, domain, lang=None): + def __init__(self, domain: str, lang: Optional[str] = None): if lang is None: lang = 'en' Node.__init__(self, @@ -1708,7 +1701,7 @@ class WebsocketCloseHeader(Node): class Features(Node): - def __init__(self, node): + def __init__(self, node: Node): Node.__init__(self, node=node) def has_starttls(self): @@ -1721,7 +1714,7 @@ class Features(Node): def has_sasl(self): return self.getTag('mechanisms', namespace=Namespace.SASL) is not None - def get_mechs(self): + def get_mechs(self) -> set[str]: mechanisms = self.getTag('mechanisms', namespace=Namespace.SASL) if mechanisms is None: return set() ===================================== nbxmpp/simplexml.py ===================================== @@ -276,7 +276,7 @@ class Node: """ if not isinstance(node, Node): node = self.getTag(node, attrs) - assert isinstance(node, Node) + self.kids.remove(node) return node ===================================== nbxmpp/structs.py ===================================== @@ -42,6 +42,7 @@ from nbxmpp.const import StatusCode from nbxmpp.const import PresenceType from nbxmpp.const import LOCATION_DATA from nbxmpp.const import AdHocStatus +from nbxmpp.const import InviteType class StanzaHandler(NamedTuple): @@ -53,17 +54,39 @@ class StanzaHandler(NamedTuple): priority: int = 50 -CommonResult = namedtuple('CommonResult', 'jid') -CommonResult.__new__.__defaults__ = (None,) +class CommonResult(NamedTuple): + jid: Optional[JID] = None -InviteData = namedtuple('InviteData', - 'muc from_ reason password type continued thread') -DeclineData = namedtuple('DeclineData', 'muc from_ reason') +class InviteData(NamedTuple): + muc: JID + from_: JID + reason: Optional[str] + password: Optional[str] + type: InviteType + continued: bool + thread: Optional[str] -CaptchaData = namedtuple('CaptchaData', 'form bob_data') -BobData = namedtuple('BobData', 'algo hash_ max_age data cid type') +class DeclineData(NamedTuple): + muc: JID + from_: JID + reason: Optional[str] + + +class CaptchaData(NamedTuple): + form: Any + bob_data: BobData + + +class BobData(NamedTuple): + algo: str + hash_: str + max_age: str + data: str + cid: str + type: str + VoiceRequest = namedtuple('VoiceRequest', 'form jid nick') ===================================== nbxmpp/util.py ===================================== @@ -15,6 +15,12 @@ # You should have received a copy of the GNU General Public License # along with this program; If not, see <http://www.gnu.org/licenses/>. +from typing import Any +from typing import Optional +from typing import Callable +from typing import Literal +from typing import Union + import base64 import hashlib import uuid @@ -35,6 +41,7 @@ from nbxmpp.namespaces import Namespace from nbxmpp.protocol import StanzaMalformed from nbxmpp.protocol import StreamHeader from nbxmpp.protocol import WebsocketOpenHeader +from nbxmpp.simplexml import Node from nbxmpp.structs import Properties from nbxmpp.structs import IqProperties from nbxmpp.structs import MessageProperties @@ -48,25 +55,24 @@ from nbxmpp.third_party.hsluv import hsluv_to_rgb log = logging.getLogger('nbxmpp.util') -def b64decode(data, return_type=str): +def b64decode(data: Union[str, bytes]) -> bytes: if not data: raise ValueError('No data to decode') + if isinstance(data, str): data = data.encode() - result = base64.b64decode(data) - if return_type == bytes: - return result - return result.decode() + return base64.b64decode(data) -def b64encode(data, return_type=str): + +def b64encode(data: Union[str, bytes]) -> str: if not data: raise ValueError('No data to encode') + if isinstance(data, str): data = data.encode() + result = base64.b64encode(data) - if return_type == bytes: - return result return result.decode() @@ -80,7 +86,7 @@ def get_properties_struct(name, own_jid): return Properties() -def from_xs_boolean(value): +def from_xs_boolean(value: str) -> bool: if value in ('1', 'true', 'True'): return True @@ -90,7 +96,7 @@ def from_xs_boolean(value): raise ValueError('Cant convert %s to python boolean' % value) -def to_xs_boolean(value): +def to_xs_boolean(value: bool) -> Literal['true', 'false']: # Convert to xs:boolean ('true', 'false') # from a python boolean (True, False) or None if value is True: @@ -110,7 +116,9 @@ error_classes = { Namespace.HTTPUPLOAD_0: HTTPUploadError } -def error_factory(stanza, condition=None, text=None): +def error_factory(stanza: Node, + condition: Optional[str] = None, + text: Optional[str] = None) -> Any: if condition == 'stanza-malformed': return StanzaMalformedError(stanza, text) app_namespace = stanza.getAppErrorNamespace() @@ -266,11 +274,11 @@ def compute_caps_hash(info, compare=True): return b64hash -def generate_id(): +def generate_id() -> str: return str(uuid.uuid4()) -def get_form(stanza, form_type): +def get_form(stanza: Node, form_type: Any) -> Any: forms = stanza.getTags('x', namespace=Namespace.DATA) if not forms: return None @@ -286,7 +294,7 @@ def get_form(stanza, form_type): return None -def validate_stream_header(stanza, domain, is_websocket): +def validate_stream_header(stanza: Node, domain: str, is_websocket: bool) -> str: attrs = stanza.getAttrs() if attrs.get('from') != domain: raise StanzaMalformed('Invalid from attr in stream header') @@ -308,18 +316,14 @@ def validate_stream_header(stanza, domain, is_websocket): return stream_id -def get_stream_header(domain, lang, is_websocket): +def get_stream_header(domain: str, lang: str, is_websocket: bool) -> str: if is_websocket: - return WebsocketOpenHeader(domain, lang) + return str(WebsocketOpenHeader(domain, lang)) header = StreamHeader(domain, lang) return "<?xml version='1.0'?>%s>" % str(header)[:-3] -def get_stanza_id(): - return str(uuid.uuid4()) - - -def utf8_decode(data): +def utf8_decode(data: bytes) -> tuple[str, bytes]: ''' Decodes utf8 byte string to unicode string Does handle invalid utf8 sequences by splitting @@ -338,7 +342,7 @@ def utf8_decode(data): raise -def get_rand_number(): +def get_rand_number() -> int: return int(binascii.hexlify(os.urandom(6)), 16) @@ -362,7 +366,7 @@ def get_invalid_xml_regex(): return re.compile(r) -def get_tls_error_phrase(tls_error): +def get_tls_error_phrase(tls_error: Gio.TlsCertificateFlags) -> str: phrase = GIO_TLS_ERRORS.get(tls_error) if phrase is None: return GIO_TLS_ERRORS.get(Gio.TlsCertificateFlags.GENERIC_ERROR) @@ -381,7 +385,7 @@ def convert_tls_error_flags(flags): return set(filter(lambda error: error & flags, GIO_TLS_ERRORS.keys())) -def get_websocket_close_string(websocket): +def get_websocket_close_string(websocket: Any) -> str: data = websocket.get_close_data() code = websocket.get_close_code() @@ -390,29 +394,29 @@ def get_websocket_close_string(websocket): return ' Data: %s Code: %s' % (data, code) -def is_websocket_close(stanza): +def is_websocket_close(stanza: Node) -> bool: return (stanza.getName() == 'close' and stanza.getNamespace() == Namespace.FRAMING) -def is_websocket_stream_error(stanza): +def is_websocket_stream_error(stanza: Node) -> bool: return (stanza.getName() == 'error' and stanza.getNamespace() == Namespace.STREAMS) class Observable: - def __init__(self, log_): + def __init__(self, log_: logging.Logger): self._log = log_ self._frozen = False - self._callbacks = defaultdict(list) + self._callbacks: defaultdict[str, list[Callable[..., Any]]] = defaultdict(list) def remove_subscriptions(self): self._callbacks = defaultdict(list) - def subscribe(self, signal_name, func): + def subscribe(self, signal_name: str, func: Callable[..., Any]): self._callbacks[signal_name].append(func) - def notify(self, signal_name, *args, **kwargs): + def notify(self, signal_name: str, *args: Any, **kwargs: dict[str, Any]): if self._frozen: self._frozen = False return View it on GitLab: https://dev.gajim.org/gajim/python-nbxmpp/-/compare/2b8b5a2262c14901d5a514c… -- View it on GitLab: https://dev.gajim.org/gajim/python-nbxmpp/-/compare/2b8b5a2262c14901d5a514c… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][mainwindow] ChatControl: Fix processing PMs
by Daniel Brötzmann (＠wurstsalat) 23 Oct '21

23 Oct '21

Daniel Brötzmann pushed to branch mainwindow at gajim / gajim Commits: 638e5bf3 by wurstsalat at 2021-10-23T18:35:45+02:00 ChatControl: Fix processing PMs - - - - - 1 changed file: - gajim/gtk/controls/base.py Changes: ===================================== gajim/gtk/controls/base.py ===================================== @@ -267,8 +267,7 @@ def process_event(self, event): if event.account != self.account: return - jid = self.contact.jid.bare - if event.jid != jid: + if event.jid not in (self.contact.jid, self.contact.jid.bare): return jingle_av_events = [ View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/638e5bf31d1bfbb2fee9453524780c9c… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/638e5bf31d1bfbb2fee9453524780c9c… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Flatpak: Remove Fcitx workaround
by André (＠andre) 23 Oct '21

23 Oct '21

André pushed to branch master at gajim / gajim Commits: 6a3fda13 by André Apitzsch at 2021-10-23T11:24:35+02:00 Flatpak: Remove Fcitx workaround For Fcitx users: If you still haven't fcitx 4.2.9.7+ installed on your system, you could run `flatpak --user override --talk-name=org.freedesktop.portal.Fcitx org.gajim.Gajim` (or `flatpak --user override --talk-name=org.freedesktop.portal.Fcitx org.gajim.Gajim.Devel`) to restore the functionality. - - - - - 1 changed file: - flatpak/org.gajim.Gajim.yaml Changes: ===================================== flatpak/org.gajim.Gajim.yaml ===================================== @@ -19,7 +19,6 @@ finish-args: - --system-talk-name=org.freedesktop.GeoClue2 - --system-talk-name=org.freedesktop.login1 - --talk-name=org.mpris.MediaPlayer2.* - - --talk-name=org.freedesktop.portal.Fcitx # Automatic status - --talk-name=org.freedesktop.ScreenSaver - --talk-name=org.gnome.Mutter.IdleMonitor View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/6a3fda13e5bd0984350dfb488b0c1b3e… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/6a3fda13e5bd0984350dfb488b0c1b3e… 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 October 2021