Commits November 2018

commits@gajim.org

1 participants
84 discussions

[Git][gajim/gajim][master] Windows: Get objdump output as string
by Philipp Hörist 13 Nov '18

13 Nov '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: 8f6dfad4 by Philipp Hörist at 2018-11-13T19:59:55Z Windows: Get objdump output as string Otherwise we have to guess the encoding of the host system - - - - - 1 changed file: - win/misc/depcheck.py Changes: ===================================== win/misc/depcheck.py ===================================== @@ -47,7 +47,7 @@ SYSTEM_LIBS = ['advapi32.dll', def get_dependencies(filename): deps = [] try: - data = subprocess.check_output(["objdump", "-p", filename]) + data = subprocess.getoutput("objdump -p %s" % filename) except Exception as error: print(error) return deps View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/8f6dfad44e482ec0267c14f6f33845d2b0… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/8f6dfad44e482ec0267c14f6f33845d2b0… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/python-nbxmpp][master] Harden parser against xml vulnerabilities
by Philipp Hörist 12 Nov '18

12 Nov '18

Philipp Hörist pushed to branch master at gajim / python-nbxmpp Commits: 70de8004 by Philipp Hörist at 2018-11-12T20:22:14Z Harden parser against xml vulnerabilities - - - - - 5 changed files: - nbxmpp/dispatcher_nb.py - nbxmpp/simplexml.py - + test/__init__.py - test/runtests.py - + test/unit/test_xml_vulnerability.py Changes: ===================================== nbxmpp/dispatcher_nb.py ===================================== @@ -235,8 +235,9 @@ class XMPPDispatcher(PlugIn): if self.Stream and self.Stream.has_received_endtag(): self._owner.disconnect(self.Stream.streamError) return 0 - except ExpatError: + except ExpatError as error: log.error('Invalid XML received from server. Forcing disconnect.') + log.error(error) self._owner.Connection.disconnect() return 0 except ValueError as e: ===================================== nbxmpp/simplexml.py ===================================== @@ -22,6 +22,7 @@ from __future__ import unicode_literals import sys import xml.parsers.expat +from xml.parsers.expat import ExpatError import logging log = logging.getLogger('nbxmpp.simplexml') @@ -566,10 +567,17 @@ class NodeBuilder(object): """ log.debug("Preparing to handle incoming XML stream.") self._parser = xml.parsers.expat.ParserCreate() + self._parser.UseForeignDTD(False) self._parser.StartElementHandler = self.starttag self._parser.EndElementHandler = self.endtag self._parser.StartNamespaceDeclHandler = self.handle_namespace_start self._parser.CharacterDataHandler = self.handle_cdata + self._parser.StartDoctypeDeclHandler = self.handle_invalid_xmpp_element + self._parser.EntityDeclHandler = self.handle_invalid_xmpp_element + self._parser.CommentHandler = self.handle_invalid_xmpp_element + self._parser.ExternalEntityRefHandler = self.handle_invalid_xmpp_element + self._parser.AttlistDeclHandler = self.handle_invalid_xmpp_element + self._parser.ProcessingInstructionHandler = self.handle_invalid_xmpp_element self._parser.buffer_text = True self.Parse = self._parser.Parse @@ -673,6 +681,10 @@ class NodeBuilder(object): self.data_buffer = [data] self.last_is_data = 1 + @staticmethod + def handle_invalid_xmpp_element(*args): + raise ExpatError('Found invalid xmpp stream element: %s' % str(args)) + def handle_namespace_start(self, prefix, uri): """ XML Parser callback. Used internally ===================================== test/__init__.py ===================================== ===================================== test/runtests.py ===================================== @@ -38,6 +38,7 @@ modules = ( 'unit.test_xmpp_dispatcher_nb', 'unit.test_xmpp_smacks', #'unit.test_xmpp_client_nb', gajim.org only supports TLS/SSL connections 'unit.test_xmpp_transports_nb2', + 'unit.test_xml_vulnerability', ) nb_errors = 0 ===================================== test/unit/test_xml_vulnerability.py ===================================== @@ -0,0 +1,73 @@ +import unittest +from unittest.mock import Mock + +from nbxmpp import dispatcher_nb + + +class XMLVulnerability(unittest.TestCase): + + def setUp(self): + self.client = Mock() + self.client.Connection = Mock() + self.dispatcher = dispatcher_nb.XMPPDispatcher() + self.dispatcher.PlugIn(self.client) + self.dispatcher.StreamInit() + + def tearDown(self): + self.dispatcher = None + self.client = None + + def test_exponential_entity_expansion(self): + bomb = """<?xml version="1.0" encoding="utf-8"?> + <!DOCTYPE bomb [ + <!ENTITY a "test"> + <!ENTITY b "&a;&a;&a;&a;&a;&a;&a;&a;&a;&a;&a;&a;&a;&a;&a;&a;"> + <!ENTITY c "&b;&b;&b;&b;&b;&b;&b;&b;&b;&b;&b;&b;&b;&b;&b;&b;"> + ]> + <bomb>&c;</bomb>""" + + self.dispatcher.ProcessNonBlocking(bomb) + self.client.Connection.disconnect.assert_called() + + def test_quadratic_blowup(self): + bomb = """<?xml version="1.0" encoding="utf-8"?> + <!DOCTYPE bomb [ + <!ENTITY a "xxxxxxx... a couple of ten thousand chars"> + ]> + <bomb>&a;&a;&a;... repeat</bomb>""" + + self.dispatcher.ProcessNonBlocking(bomb) + self.client.Connection.disconnect.assert_called() + + def test_external_entity_expansion(self): + bomb = """<?xml version="1.0" encoding="utf-8"?> + <!DOCTYPE external [ + <!ENTITY ee SYSTEM "http://www.python.org/some.xml"> + ]> + <root>&ee;</root>""" + + self.dispatcher.ProcessNonBlocking(bomb) + self.client.Connection.disconnect.assert_called() + + def test_external_local_entity_expansion(self): + bomb = """<?xml version="1.0" encoding="utf-8"?> + <stream:stream xmlns:stream='http://etherx.jabber.org/streams' xmlns='jabber:client'> + <!DOCTYPE external [ + <!ENTITY ee SYSTEM "file:///PATH/TO/simple.xml"> + ]> + <root>&ee;</root>""" + + self.dispatcher.ProcessNonBlocking(bomb) + self.client.Connection.disconnect.assert_called() + + def test_dtd_retrival(self): + bomb = """<?xml version="1.0" encoding="utf-8"?> + <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" + "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> + <html> + <head/> + <body>text</body> + </html>""" + + self.dispatcher.ProcessNonBlocking(bomb) + self.client.Connection.disconnect.assert_called() View it on GitLab: https://dev.gajim.org/gajim/python-nbxmpp/commit/70de8004a84ef62e0e2c25bc1c… -- View it on GitLab: https://dev.gajim.org/gajim/python-nbxmpp/commit/70de8004a84ef62e0e2c25bc1c… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Correctly remove placeholder on focus in
by Philipp Hörist 12 Nov '18

12 Nov '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: 9c5de3bf by Philipp Hörist at 2018-11-12T16:47:04Z Correctly remove placeholder on focus in This is a regression from e281fb68 Fixes #9440 - - - - - 1 changed file: - gajim/message_textview.py Changes: ===================================== gajim/message_textview.py ===================================== @@ -98,6 +98,7 @@ class MessageTextView(Gtk.TextView): self.connect('paste-clipboard', self._paste_clipboard) self.connect_after('paste-clipboard', self._after_paste_clipboard) + self.connect('focus-in-event', self._on_grab_focus) self.connect('grab-focus', self._on_grab_focus) self.connect('focus-out-event', self._on_focus_out) View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/9c5de3bf25cd2183bd70a813da99772f8b… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/9c5de3bf25cd2183bd70a813da99772f8b… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Refactor htmltextview test
by Philipp Hörist 11 Nov '18

11 Nov '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: f26db884 by Philipp Hörist at 2018-11-11T21:54:43Z Refactor htmltextview test - - - - - 1 changed file: - test/gtk/htmltextview.py Changes: ===================================== test/gtk/htmltextview.py ===================================== @@ -1,17 +1,14 @@ -import sys -import os from unittest.mock import MagicMock -sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..', '..'))) - -from gi.repository import Gtk, Gdk +import gi +gi.require_version('Gtk', '3.0') +from gi.repository import Gtk from gajim.common import app from gajim.common import configpaths configpaths.init() from gajim.common import caps_cache -from gajim.gtk.util import get_cursor from gajim.conversation_textview import ConversationTextview from gajim.gui_interface import Interface @@ -20,168 +17,192 @@ app.plugin_manager = MagicMock() app.logger = MagicMock() app.interface = Interface() -change_cursor = None -htmlview = ConversationTextview(None) - - -def on_textview_motion_notify_event(widget, event): - """ - Change the cursor to a hand when we are over a mail or an url - """ - global change_cursor - w = htmlview.tv.get_window(Gtk.TextWindowType.TEXT) - device = w.get_display().get_device_manager().get_client_pointer() - pointer = w.get_device_position(device) - x = pointer[1] - y = pointer[2] - tags = htmlview.tv.get_iter_at_location(x, y)[1].get_tags() - if change_cursor: - w.set_cursor(get_cursor('XTERM')) - change_cursor = None - for tag in tags: - try: - if tag.is_anchor: - w.set_cursor(get_cursor('HAND2')) - change_cursor = tag - except Exception: - pass - -htmlview.tv.connect('motion_notify_event', on_textview_motion_notify_event) - -def handler(texttag, widget, event, iter_, kind): - if event.type == Gdk.EventType.BUTTON_PRESS: - pass - - -htmlview.print_real_text(None, xhtml='<div>' -'Hello \n' - ' <img src="http://images.slashdot.org/topics/topicsoftware.gif"/> \n' -'World\n' - '</div>\n') -htmlview.print_real_text(None, xhtml='<hr />') -htmlview.print_real_text(None, xhtml=''' -<body xmlns='http://www.w3.org/1999/xhtml'> - a:b - <a href='http://google.com/' xmlns='http://www.w3.org/1999/xhtml'>Google - </a> - -</body>''') -htmlview.print_real_text(None, xhtml=''' - <body xmlns='http://www.w3.org/1999/xhtml'> - - O - MG, - I'm green - with envy! - - </body> - ''') -htmlview.print_real_text(None, xhtml='<hr />') -htmlview.print_real_text(None, xhtml=''' -<body xmlns='http://www.w3.org/1999/xhtml'> + +XHTML = [ + ''' + <div> + Hello + \n + <img src="http://images.slashdot.org/topics/topicsoftware.gif"/> + \n + World\n + </div> + \n + ''', + + ''' + <hr /> + ''', + + ''' + <body xmlns='http://www.w3.org/1999/xhtml'> + a:b + <a href='http://google.com/' xmlns='http://www.w3.org/1999/xhtml'>Google</a> + + + </body> + ''', + + ''' + <body xmlns='http://www.w3.org/1999/xhtml'> + + O + MG + , I'm greenwith + envy! + + </body> + ''', + + ''' + <hr /> + ''', + + ''' + <body xmlns='http://www.w3.org/1999/xhtml'> http://test.com/ testing links autolinkifying -</body> - ''') -htmlview.print_real_text(None, xhtml='<hr />') -htmlview.print_real_text(None, xhtml=''' -<body xmlns='http://www.w3.org/1999/xhtml'> - As Emerson said in his essay Self-Reliance: - - "A foolish consistency is the hobgoblin of little minds." - -</body> - ''') -htmlview.print_real_text(None, xhtml='<hr />') -htmlview.print_real_text(None, xhtml=''' -<body xmlns='http://www.w3.org/1999/xhtml'> - - Hey, are you licensed to <a href='http://www.jabber.org/'>Jabber</a>? - - - <img src='http://www.xmpp.org/images/psa-license.jpg' - alt='A License to Jabber' width='50%' height='50%'/> - -</body> - ''') -htmlview.print_real_text(None, xhtml='<hr />') -htmlview.print_real_text(None, xhtml=''' -<body xmlns='http://www.w3.org/1999/xhtml'> - <ul style='background-color:rgb(120,140,100)'> - <li> One </li> - <li> Two </li> - <li> Three </li> - </ul><hr /><pre style="background-color:rgb(120,120,120)">def fac(n): -def faciter(n,acc): -if n==0: return acc -return faciter(n-1, acc*n) -if n<0: raise ValueError('Must be non-negative') -return faciter(n,1)</pre> -</body> - ''') -htmlview.print_real_text(None, xhtml='<hr />') -htmlview.print_real_text(None, xhtml=''' -<body xmlns='http://www.w3.org/1999/xhtml'> - <ol style='background-color:rgb(120,140,100)'> - <li> One </li> - <li> Two is nested: <ul style='background-color:rgb(200,200,100)'> - <li> One </li> - <li style='font-size:50%'> Two </li> - <li style='font-size:200%'> Three </li> - <li style='font-size:9999pt'> Four </li> - </ul></li> - <li> Three </li></ol> -</body> - ''') -htmlview.print_real_text(None, xhtml='<hr />') -htmlview.print_real_text(None, xhtml=''' -<body xmlns='http://www.w3.org/1999/xhtml'> - - - <a href='xmpp:example@example.org'>xmpp link</a> - : -<div xmlns='http://www.w3.org/1999/xhtml'> - <cite style='margin: 7px;' title='xmpp:examples@example.org'> - - examples(a)example.org wrote: - - this cite - bla bla bla, smile- :-) … - </cite> - <div> - some text - </div> -</div> - -#232/1 -</body> -''') -htmlview.print_real_text(None, xhtml='<hr />') -htmlview.print_real_text(None, xhtml=''' -<body xmlns='http://www.w3.org/1999/xhtml'> - -<img src='data:image/png;base64,R0lGODdhMAAwAPAAAAAAAP///ywAAAAAMAAw\ -AAAC8IyPqcvt3wCcDkiLc7C0qwyGHhSWpjQu5yqmCYsapyuvUUlvONmOZtfzgFz\ -ByTB10QgxOR0TqBQejhRNzOfkVJ+5YiUqrXF5Y5lKh/DeuNcP5yLWGsEbtLiOSp\ -a/TPg7JpJHxyendzWTBfX0cxOnKPjgBzi4diinWGdkF8kjdfnycQZXZeYGejmJl\ -ZeGl9i2icVqaNVailT6F5iJ90m6mvuTS4OK05M0vDk0Q4XUtwvKOzrcd3iq9uis\ -F81M1OIcR7lEewwcLp7tuNNkM3uNna3F2JQFo97Vriy/Xl4/f1cf5VWzXyym7PH\ -hhx4dbgYKAAA7' alt='Larry'/> -</body> -''') -htmlview.tv.show() -sw = Gtk.ScrolledWindow() -sw.set_property('hscrollbar-policy', Gtk.PolicyType.AUTOMATIC) -sw.set_property('vscrollbar-policy', Gtk.PolicyType.AUTOMATIC) -sw.set_property('border-width', 0) -sw.add(htmlview.tv) -sw.show() -frame = Gtk.Frame() -frame.set_shadow_type(Gtk.ShadowType.IN) -frame.show() -frame.add(sw) -win = Gtk.Window() -win.add(frame) -win.set_default_size(400, 300) + </body> + ''', + + ''' + <hr /> + ''', + + ''' + <body xmlns='http://www.w3.org/1999/xhtml'> + As Emerson said in his essay Self-Reliance: + + "A foolish consistency is the hobgoblin of little minds." + + </body> + ''', + + ''' + <hr /> + ''', + + ''' + <body xmlns='http://www.w3.org/1999/xhtml'> + + Hey, are you licensed to <a href='http://www.jabber.org/'>Jabber</a>? + + + <img src='http://www.xmpp.org/images/psa-license.jpg' + alt='A License to Jabber' width='50%' height='50%'/> + + </body> + ''', + + ''' + <hr /> + ''', + + ''' + <body xmlns='http://www.w3.org/1999/xhtml'> + <ul style='background-color:rgb(120,140,100)'> + <li> One </li> + <li> Two </li> + <li> Three </li> + </ul> + <hr /> + <pre style="background-color:rgb(120,120,120)">def fac(n): + def faciter(n,acc): + if n==0: return acc + return faciter(n-1, acc*n) + if n<0: raise ValueError('Must be non-negative') + return faciter(n,1)</pre> + </body> + ''', + + ''' + <hr /> + ''', + + ''' + <body xmlns='http://www.w3.org/1999/xhtml'> + <ol style='background-color:rgb(120,140,100)'> + <li> One </li> + <li> + Two is nested: + <ul style='background-color:rgb(200,200,100)'> + <li> One </li> + <li style='font-size:50%'> Two </li> + <li style='font-size:200%'> Three </li> + <li style='font-size:9999pt'> Four </li> + </ul> + </li> + <li> Three </li> + </ol> + </body> + ''', + + ''' + <hr /> + ''', + + ''' + <body xmlns='http://www.w3.org/1999/xhtml'> + + + <a href='xmpp:example@example.org'>xmpp link</a> + : + + <div xmlns='http://www.w3.org/1999/xhtml'> + <cite style='margin: 7px;' title='xmpp:examples@example.org'> + + examples(a)example.org wrote: + + this cite - bla bla bla, smile- :-) … + </cite> + <div> + some text + </div> + </div> + + #232/1 + </body> + ''', + + ''' + <hr /> + ''', + + ''' + <body xmlns='http://www.w3.org/1999/xhtml'> + + <img src='data:image/png;base64,R0lGODdhMAAwAPAAAAAAAP///ywAAAAAMAAw\ + AAAC8IyPqcvt3wCcDkiLc7C0qwyGHhSWpjQu5yqmCYsapyuvUUlvONmOZtfzgFz\ + ByTB10QgxOR0TqBQejhRNzOfkVJ+5YiUqrXF5Y5lKh/DeuNcP5yLWGsEbtLiOSp\ + a/TPg7JpJHxyendzWTBfX0cxOnKPjgBzi4diinWGdkF8kjdfnycQZXZeYGejmJl\ + ZeGl9i2icVqaNVailT6F5iJ90m6mvuTS4OK05M0vDk0Q4XUtwvKOzrcd3iq9uis\ + F81M1OIcR7lEewwcLp7tuNNkM3uNna3F2JQFo97Vriy/Xl4/f1cf5VWzXyym7PH\ + hhx4dbgYKAAA7' alt='Larry'/> + </body> + ''', + +] + + +class TextviewWindow(Gtk.Window): + def __init__(self): + Gtk.Window.__init__(self, title="Textview Test") + self.set_default_size(600, 600) + + self._textview = ConversationTextview(None) + + scrolled = Gtk.ScrolledWindow() + scrolled.set_policy(Gtk.PolicyType.AUTOMATIC, Gtk.PolicyType.AUTOMATIC) + scrolled.add(self._textview.tv) + self.add(scrolled) + self.show() + self._print_xhtml() + + def _print_xhtml(self): + for xhtml in XHTML: + self._textview.print_real_text(None, xhtml=xhtml) + +win = TextviewWindow() +win.connect("destroy", Gtk.main_quit) win.show_all() -win.connect('destroy', lambda win: Gtk.main_quit()) Gtk.main() View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/f26db88424f000a9dde21039f9a5f6a180… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/f26db88424f000a9dde21039f9a5f6a180… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Fix clipboard paste in inactive window
by Philipp Hörist 11 Nov '18

11 Nov '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: e281fb68 by Andrey Gursky at 2018-11-11T16:20:36Z Fix clipboard paste in inactive window Fixes #9427 - - - - - 1 changed file: - gajim/message_textview.py Changes: ===================================== gajim/message_textview.py ===================================== @@ -98,13 +98,9 @@ class MessageTextView(Gtk.TextView): self.connect('paste-clipboard', self._paste_clipboard) self.connect_after('paste-clipboard', self._after_paste_clipboard) - self.connect('focus-in-event', self._on_focus_in) + self.connect('grab-focus', self._on_grab_focus) self.connect('focus-out-event', self._on_focus_out) - start = buffer_.get_bounds()[0] - buffer_.insert_with_tags( - start, self.PLACEHOLDER, self.placeholder_tag) - def _on_buffer_changed(self, *args): text = self.get_text() # Because of inserting and removing PLACEHOLDER text @@ -136,11 +132,14 @@ class MessageTextView(Gtk.TextView): text = buf.get_text(start, end, True) return text == self.PLACEHOLDER - def _on_focus_in(self, *args): + def remove_placeholder(self): if self.is_placeholder(): self.get_buffer().set_text('') self.toggle_speller(True) + def _on_grab_focus(self, *args): + self.remove_placeholder() + def _on_focus_out(self, *args): buf = self.get_buffer() start, end = buf.get_bounds() @@ -155,9 +154,6 @@ class MessageTextView(Gtk.TextView): spell_view = Gspell.TextView.get_from_gtk_text_view(self) spell_view.set_inline_spell_checking(activate) - def remove_placeholder(self): - self._on_focus_in() - @staticmethod def _paste_clipboard(textview): if textview.is_placeholder(): View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/e281fb6839034f904f49c787bbbd5e17cb… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/e281fb6839034f904f49c787bbbd5e17cb… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] 3 commits: Test: remove unused imports
by Philipp Hörist 11 Nov '18

11 Nov '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: 49758ab3 by André Apitzsch at 2018-11-11T15:50:59Z Test: remove unused imports - - - - - 522eec00 by André Apitzsch at 2018-11-11T15:50:59Z Test: caps cache: use mock from unittest - - - - - 989926a6 by André Apitzsch at 2018-11-11T15:50:59Z Restructure test - - - - - 21 changed files: - .gitlab-ci.yml - setup.cfg - setup.py - test/integration/test_gui_event_integration.py → test/broken/integration/test_gui_event_integration.py - test/integration/test_roster.py → test/broken/integration/test_roster.py - test/test_pluginmanager.py → test/broken/test_pluginmanager.py - test/unit/test_jingle.py → test/broken/unit/test_jingle.py - test/unit/test_sessions.py → test/broken/unit/test_sessions.py - test/unit/test_socks5.py → test/broken/unit/test_socks5.py - test/gtk/htmltextview.py - test/integration/test_resolver.py - test/lib/__init__.py - test/lib/gajim_mocks.py - + test/no_gui/__init__.py - + test/no_gui/unit/__init__.py - test/unit/test_account.py → test/no_gui/unit/test_account.py - test/unit/test_caps_cache.py → test/no_gui/unit/test_caps_cache.py - test/unit/test_contacts.py → test/no_gui/unit/test_contacts.py - test/unit/test_protocol_caps.py → test/no_gui/unit/test_protocol_caps.py - − test/runtests.py - test/unit/test_gui_interface.py Changes: ===================================== .gitlab-ci.yml ===================================== @@ -10,7 +10,7 @@ stages: run-test: stage: test script: - - python3 setup.py test_nogui + - python3 setup.py test -s test.no_gui -q run-mypy: stage: test ===================================== setup.cfg ===================================== @@ -18,6 +18,7 @@ classifiers = [options] python_requires = >=3.5 packages = find: +test_suite = test install_requires = cssutils>=1.0.2 keyring ===================================== setup.py ===================================== @@ -176,27 +176,6 @@ class install(_install): _install.run(self) -class test(Command): - description = "Run all tests" - user_options = [] - - def initialize_options(self): - pass - - def finalize_options(self): - pass - - def run(self): - exit(subprocess.call("./test/runtests.py")) - - -class test_nogui(test): - description = "Run tests without GUI" - - def run(self): - exit(subprocess.call(["./test/runtests.py", "-n"])) - - class update_po(Command): description = "Update po files" user_options = [] @@ -229,8 +208,6 @@ setup( cmdclass={ 'build_py': build, 'install': install, - 'test': test, - 'test_nogui': test_nogui, 'update_po': update_po, }, entry_points={ ===================================== test/integration/test_gui_event_integration.py → test/broken/integration/test_gui_event_integration.py ===================================== ===================================== test/integration/test_roster.py → test/broken/integration/test_roster.py ===================================== @@ -5,7 +5,6 @@ lib.setup_env() from data import * -from mock import Mock, expectParams from gajim_mocks import * from gajim.common import app ===================================== test/test_pluginmanager.py → test/broken/test_pluginmanager.py ===================================== ===================================== test/unit/test_jingle.py → test/broken/unit/test_jingle.py ===================================== ===================================== test/unit/test_sessions.py → test/broken/unit/test_sessions.py ===================================== @@ -1,12 +1,10 @@ import unittest -import time - import lib lib.setup_env() - import notify +import nbxmpp from gajim.common import app from gajim.common import nec @@ -14,7 +12,6 @@ from gajim.common import ged from gajim.common.nec import NetworkEvent from gajim.common.modules.message import MessageReceivedEvent from gajim.common.modules.message import DecryptedMessageReceivedEvent -import nbxmpp from gajim.session import ChatControlSession from gajim.roster_window import RosterWindow @@ -54,8 +51,6 @@ class TestChatControlSession(unittest.TestCase): msg.setBody(msgtxt) msg.setType('chat') - tim = time.localtime() - encrypted = False xml = """<message from='%s' id='1' type='chat'><body>%s</body> <thread>123</thread></message>""" % (jid, msgtxt) stanza = nbxmpp.protocol.Message(node=nbxmpp.simplexml.XML2Node(xml)) ===================================== test/unit/test_socks5.py → test/broken/unit/test_socks5.py ===================================== @@ -167,8 +167,6 @@ class TestSocks5(unittest.TestCase): self._check_inout() - - if __name__ == '__main__': unittest.main() ===================================== test/gtk/htmltextview.py ===================================== @@ -15,7 +15,6 @@ from gajim.gtk.util import get_cursor from gajim.conversation_textview import ConversationTextview from gajim.gui_interface import Interface - caps_cache.capscache = MagicMock() app.plugin_manager = MagicMock() app.logger = MagicMock() @@ -24,6 +23,7 @@ app.interface = Interface() change_cursor = None htmlview = ConversationTextview(None) + def on_textview_motion_notify_event(widget, event): """ Change the cursor to a hand when we are over a mail or an url @@ -52,7 +52,6 @@ def handler(texttag, widget, event, iter_, kind): if event.type == Gdk.EventType.BUTTON_PRESS: pass -htmlview.tv.hyperlink_handler = htmlview.hyperlink_handler htmlview.print_real_text(None, xhtml='<div>' 'Hello \n' ===================================== test/integration/test_resolver.py ===================================== @@ -1,22 +1,19 @@ import unittest -import time - -import lib +from test import lib lib.setup_env() from gi.repository import GLib from gajim.common import resolver -from gajim_mocks import * - NONSENSE_NAME = 'sfsdfsdfsdf.sdfs.fsd' JABBERCZ_TXT_NAME = '_xmppconnect.jabber.cz' JABBERCZ_SRV_NAME = '_xmpp-client._tcp.jabber.cz' TEST_LIST = [(NONSENSE_NAME, 'srv', False), - (JABBERCZ_SRV_NAME, 'srv', True)] + (JABBERCZ_SRV_NAME, 'srv', True)] + class TestResolver(unittest.TestCase): ''' @@ -47,9 +44,9 @@ class TestResolver(unittest.TestCase): def _runGR(self, name, type_): self.resolver.resolve( - host = name, - type_ = type_, - on_ready = self._myonready) + host=name, + type_=type_, + on_ready=self._myonready) self.main_loop.run() ===================================== test/lib/__init__.py ===================================== @@ -1,26 +1,15 @@ -import sys import os -import getopt -use_x = True -shortargs = 'hnv:' -longargs = 'help no-x verbose=' -opts, args = getopt.getopt(sys.argv[1:], shortargs, longargs.split()) -for o, a in opts: - if o in ('-n', '--no-x'): - use_x = False - -gajim_root = os.path.join(os.path.abspath(os.path.dirname(__file__)), '../..') - -# look for modules in the CWD, then gajim/test/lib, then gajim/gajim, -# then everywhere else -sys.path.insert(1, gajim_root) -sys.path.insert(1, gajim_root + '/test/lib') +from tempfile import gettempdir # a temporary version of ~/.gajim for testing -configdir = gajim_root + '/test/tmp' +configdir = os.path.join(gettempdir(), 'gajim') +os.makedirs(configdir, exist_ok=True) + # plugins config dir pluginsconfigdir = configdir + '/pluginsconfig' +# theme config directory +themedir = configdir + '/theme' # define _ for i18n import builtins @@ -28,7 +17,7 @@ builtins._ = lambda x: x from gajim.common.contacts import LegacyContactsAPI -def setup_env(): +def setup_env(use_x=True): # wipe config directory if os.path.isdir(configdir): import shutil @@ -36,6 +25,7 @@ def setup_env(): os.mkdir(configdir) os.mkdir(pluginsconfigdir) + os.mkdir(themedir) from gajim.common import configpaths configpaths.set_config_root(configdir) @@ -47,13 +37,10 @@ def setup_env(): import logging logging.basicConfig() - configpaths.override_path('DATA', gajim_root + '/gajim/data') app.use_x = use_x app.contacts = LegacyContactsAPI() app.connections = {} if use_x: - from gajim import gtkgui_helpers - gtkgui_helpers.GUI_DIR = gajim_root + '/gajim/data/gui' from gajim.application import GajimApplication app.app = GajimApplication() ===================================== test/lib/gajim_mocks.py ===================================== @@ -2,7 +2,7 @@ Module with dummy classes for Gajim specific unit testing ''' -from mock import Mock +from .mock import Mock from gajim.common import app from gajim.common import ged ===================================== test/no_gui/__init__.py ===================================== ===================================== test/no_gui/unit/__init__.py ===================================== @@ -0,0 +1,5 @@ +''' + +This package just contains plain unit tests + +''' ===================================== test/unit/test_account.py → test/no_gui/unit/test_account.py ===================================== @@ -3,11 +3,9 @@ Tests for Account classes ''' import unittest -import lib -lib.setup_env() - from gajim.common.account import Account + class Test(unittest.TestCase): def testInstantiate(self): ===================================== test/unit/test_caps_cache.py → test/no_gui/unit/test_caps_cache.py ===================================== @@ -3,16 +3,11 @@ Tests for capabilities and the capabilities cache ''' import unittest -import lib -lib.setup_env() - -from unittest.mock import MagicMock +from unittest.mock import MagicMock, Mock from nbxmpp import NS_MUC, NS_PING, NS_XHTML_IM, Iq from gajim.common import caps_cache as caps from gajim.common.modules.discovery import Discovery -from mock import Mock - COMPLEX_EXAMPLE = ''' <iq from='benvolio(a)capulet.lit/230193' id='disco1' to='juliet(a)capulet.lit/chamber' type='result'> <query xmlns='http://jabber.org/protocol/disco#info' node='http://psi-im.org#q07IKJEyjvHSyhy//CH0CxmKi8w='> @@ -64,7 +59,8 @@ class CommonCapsTest(unittest.TestCase): db_caps_cache = [ (self.caps_method, self.caps_hash, self.identities, self.features), ('old', self.node + '#' + self.caps_hash, self.identities, self.features)] - self.logger = Mock(returnValues={"iter_caps_data":db_caps_cache}) + self.logger = Mock() + self.logger.iter_caps_data = Mock(return_value=db_caps_cache) self.cc = caps.CapsCache(self.logger) caps.capscache = self.cc @@ -95,8 +91,10 @@ class TestCapsCache(CommonCapsTest): item = self.cc[self.client_caps] item.set_and_store(self.identities, self.features) - self.logger.mockCheckCall(0, "add_caps_entry", self.caps_method, - self.caps_hash, self.identities, self.features) + self.logger.add_caps_entry.assert_called_once_with(self.caps_method, + self.caps_hash, + self.identities, + self.features) def test_initialize_from_db(self): ''' Read cashed dummy data from db ''' ===================================== test/unit/test_contacts.py → test/no_gui/unit/test_contacts.py ===================================== @@ -2,12 +2,9 @@ Test for Contact, GC_Contact and Contacts ''' import unittest - -import lib -lib.setup_env() +from nbxmpp import NS_MUC from gajim.common.contacts import CommonContact, Contact, GC_Contact, LegacyContactsAPI -from nbxmpp import NS_MUC from gajim.common import caps_cache ===================================== test/unit/test_protocol_caps.py → test/no_gui/unit/test_protocol_caps.py ===================================== @@ -7,9 +7,6 @@ from unittest.mock import MagicMock import nbxmpp -import lib -lib.setup_env() - from gajim.common import app from gajim.common import nec from gajim.common import ged ===================================== test/runtests.py deleted ===================================== @@ -1,71 +0,0 @@ -#!/usr/bin/env python3 - - -''' -Runs Gajim's Test Suite - -Unit tests tests will be run on each commit. -''' - -import sys -import unittest -import getopt -use_x = True -verbose = 1 - -try: - shortargs = 'hnv:' - longargs = 'help no-x verbose=' - opts, args = getopt.getopt(sys.argv[1:], shortargs, longargs.split()) -except getopt.error as msg: - print(msg) - print('for help use --help') - sys.exit(2) -for o, a in opts: - if o in ('-h', '--help'): - print('runtests [--help] [--no-x] [--verbose level]') - sys.exit() - elif o in ('-n', '--no-x'): - use_x = False - elif o in ('-v', '--verbose'): - try: - verbose = int(a) - except Exception: - print('verbose must be a number >= 0') - sys.exit(2) - -# new test modules need to be added manually -modules = ( 'unit.test_protocol_caps', - 'unit.test_caps_cache', - 'unit.test_contacts', - 'unit.test_account', - ) - -if use_x: - modules += ( 'unit.test_sessions', - #'integration.test_gui_event_integration', - 'integration.test_roster', - 'integration.test_resolver', - 'unit.test_gui_interface', - ) - -nb_errors = 0 -nb_failures = 0 - -for mod in modules: - print("Now running: %s" % mod) - suite = unittest.defaultTestLoader.loadTestsFromName(mod) - result = unittest.TextTestRunner(verbosity=verbose).run(suite) - if use_x: - # Wait 500ms to be sure all timeout_add will be called before we cleanup main loop - import time - time.sleep(0.5) - # Clean main loop - from gi.repository import GLib - mc = GLib.main_context_default() - while mc.pending(): - mc.iteration() - nb_errors += len(result.errors) - nb_failures += len(result.failures) - -sys.exit(nb_errors + nb_failures) ===================================== test/unit/test_gui_interface.py ===================================== @@ -3,7 +3,7 @@ Some diverse tests covering functionality in the GUI Interface class. ''' import unittest -import lib +from test import lib lib.setup_env() from gajim.common import logging_helpers @@ -11,9 +11,6 @@ logging_helpers.set_quiet() from gajim.common import app -from gajim_mocks import MockLogger -app.logger = MockLogger() - from gajim.gui_interface import Interface from gi.repository import GLib View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/5ecdfbee5c4ba4e084df4dc6b291c5a6a… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/5ecdfbee5c4ba4e084df4dc6b291c5a6a… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Windows: Remove old ssl fix
by Philipp Hörist 11 Nov '18

11 Nov '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: 5ecdfbee by Philipp Hörist at 2018-11-11T15:40:08Z Windows: Remove old ssl fix - - - - - 2 changed files: - win/_base.sh - win/build.sh Changes: ===================================== win/_base.sh ===================================== @@ -302,19 +302,6 @@ function cleanup_install { } -function download_ssl_libs { - # Python needs these ssl libs but msys does not provide them - curl -o "${BUILD_ROOT}"/ssl.zip https://gajim.org/downloads/snap/win/build/ssl.zip - 7z x -o"${MINGW_ROOT}"/bin "${BUILD_ROOT}"/ssl.zip -} - -function move_ssl_libs { - # Pythons ssl module searches in that path for these dlls, if they are not there - # C:/Windows/system32 is searcherd and potentially wrong versioned dlls are found there - cp "${MINGW_ROOT}"/bin/libeay32.dll "${MINGW_ROOT}"/lib/python3.7/lib-dynload/libeay32.dll - cp "${MINGW_ROOT}"/bin/ssleay32.dll "${MINGW_ROOT}"/lib/python3.7/lib-dynload/ssleay32.dll -} - function build_installer { (cd "$BUILD_ROOT" && makensis -NOCD -DVERSION="$QL_VERSION_DESC" -DARCH="${MINGW}" "${MISC}"/gajim.nsi) (cd "$BUILD_ROOT" && makensis -NOCD -DVERSION="$QL_VERSION_DESC" -DARCH="${MINGW}" "${MISC}"/gajim-portable.nsi) ===================================== win/build.sh ===================================== @@ -17,8 +17,6 @@ function main { install_deps install_gajim cleanup_install - download_ssl_libs - move_ssl_libs build_installer } View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/5ecdfbee5c4ba4e084df4dc6b291c5a6ad… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/5ecdfbee5c4ba4e084df4dc6b291c5a6ad… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][gajim_1.1] Windows: Fix build
by Philipp Hörist 11 Nov '18

11 Nov '18

Philipp Hörist pushed to branch gajim_1.1 at gajim / gajim Commits: 335c3e2d by Philipp Hörist at 2018-11-11T14:41:12Z Windows: Fix build Fixes #9438 - - - - - 1 changed file: - win/_base.sh Changes: ===================================== win/_base.sh ===================================== @@ -71,9 +71,14 @@ function create_root { function install_deps { # Downgrade GLIB, resolver is broken on Windows 7 - curl -o "${BUILD_ROOT}"/mingw-w64-x86_64-glib2-2.56.2-1-any.pkg.tar.xz \ - https://gajim.org/downloads/snap/win/build/mingw-w64-x86_64-glib2-2.56.2-1-… - build_pacman --noconfirm -U "${BUILD_ROOT}"/mingw-w64-x86_64-glib2-2.56.2-1-any.pkg.tar.xz + curl -o "${BUILD_ROOT}"/mingw-w64-"${ARCH}"-glib2-2.56.2-1-any.pkg.tar.xz \ + https://gajim.org/downloads/snap/win/build/mingw-w64-"${ARCH}"-glib2-2.56.2-1-any.pkg.tar.xz + build_pacman --noconfirm -U "${BUILD_ROOT}"/mingw-w64-"${ARCH}"-glib2-2.56.2-1-any.pkg.tar.xz + + # Fix broken cryptography package + curl -o "${BUILD_ROOT}"/mingw-w64-"${ARCH}"-python3-cryptography-2.3.1-1-any.pkg.tar.xz \ + https://gajim.org/downloads/snap/win/build/mingw-w64-"${ARCH}"-python3-cryptography-2.3.1-1-any.pkg.tar.xz + build_pacman --noconfirm -U "${BUILD_ROOT}"/mingw-w64-"${ARCH}"-python3-cryptography-2.3.1-1-any.pkg.tar.xz build_pacman --noconfirm -S mingw-w64-"${ARCH}"-gtk3 mingw-w64-"${ARCH}"-"${PYTHON_ID}" \ mingw-w64-"${ARCH}"-"${PYTHON_ID}"-gobject \ @@ -87,7 +92,6 @@ function install_deps { mingw-w64-"${ARCH}"-"${PYTHON_ID}"-setuptools \ mingw-w64-"${ARCH}"-"${PYTHON_ID}"-pillow \ mingw-w64-"${ARCH}"-"${PYTHON_ID}"-setuptools-scm \ - mingw-w64-"${ARCH}"-"${PYTHON_ID}"-cryptography \ mingw-w64-"${ARCH}"-"${PYTHON_ID}"-pyopenssl \ mingw-w64-"${ARCH}"-"${PYTHON_ID}"-docutils \ mingw-w64-"${ARCH}"-"${PYTHON_ID}"-certifi \ View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/335c3e2d3df71fb07a6d96e9c0b6ae3db0… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/335c3e2d3df71fb07a6d96e9c0b6ae3db0… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] 2 commits: Determine absolut path for custom config folder
by Philipp Hörist 11 Nov '18

11 Nov '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: e509a681 by Philipp Hörist at 2018-11-11T14:15:10Z Determine absolut path for custom config folder When a relative custom config path is passed via commandline, the logic which decides if a plugin can be deleted from the plugin manager fails - - - - - ac5bfe2d by Philipp Hörist at 2018-11-11T14:37:30Z Windows: Fix build Fixes #9438 - - - - - 2 changed files: - gajim/common/configpaths.py - win/_base.sh Changes: ===================================== gajim/common/configpaths.py ===================================== @@ -69,7 +69,7 @@ def set_profile(profile: str) -> None: def set_config_root(config_root: str) -> None: - _paths.custom_config_root = config_root + _paths.custom_config_root = str(Path(config_root).resolve()) def init() -> None: ===================================== win/_base.sh ===================================== @@ -71,9 +71,14 @@ function create_root { function install_deps { # Downgrade GLIB, resolver is broken on Windows 7 - curl -o "${BUILD_ROOT}"/mingw-w64-x86_64-glib2-2.56.2-1-any.pkg.tar.xz \ - https://gajim.org/downloads/snap/win/build/mingw-w64-x86_64-glib2-2.56.2-1-… - build_pacman --noconfirm -U "${BUILD_ROOT}"/mingw-w64-x86_64-glib2-2.56.2-1-any.pkg.tar.xz + curl -o "${BUILD_ROOT}"/mingw-w64-"${ARCH}"-glib2-2.56.2-1-any.pkg.tar.xz \ + https://gajim.org/downloads/snap/win/build/mingw-w64-"${ARCH}"-glib2-2.56.2-1-any.pkg.tar.xz + build_pacman --noconfirm -U "${BUILD_ROOT}"/mingw-w64-"${ARCH}"-glib2-2.56.2-1-any.pkg.tar.xz + + # Fix broken cryptography package + curl -o "${BUILD_ROOT}"/mingw-w64-"${ARCH}"-python3-cryptography-2.3.1-1-any.pkg.tar.xz \ + https://gajim.org/downloads/snap/win/build/mingw-w64-"${ARCH}"-python3-cryptography-2.3.1-1-any.pkg.tar.xz + build_pacman --noconfirm -U "${BUILD_ROOT}"/mingw-w64-"${ARCH}"-python3-cryptography-2.3.1-1-any.pkg.tar.xz build_pacman --noconfirm -S mingw-w64-"${ARCH}"-gtk3 mingw-w64-"${ARCH}"-"${PYTHON_ID}" \ mingw-w64-"${ARCH}"-"${PYTHON_ID}"-gobject \ @@ -87,7 +92,6 @@ function install_deps { mingw-w64-"${ARCH}"-"${PYTHON_ID}"-setuptools \ mingw-w64-"${ARCH}"-"${PYTHON_ID}"-pillow \ mingw-w64-"${ARCH}"-"${PYTHON_ID}"-setuptools-scm \ - mingw-w64-"${ARCH}"-"${PYTHON_ID}"-cryptography \ mingw-w64-"${ARCH}"-"${PYTHON_ID}"-pyopenssl \ mingw-w64-"${ARCH}"-"${PYTHON_ID}"-docutils \ mingw-w64-"${ARCH}"-"${PYTHON_ID}"-certifi \ View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/62b210a9e5db79413ca1eef90b74698d3… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/62b210a9e5db79413ca1eef90b74698d3… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Pass account to plugins when encrypting a file
by Philipp Hörist 11 Nov '18

11 Nov '18

Philipp Hörist pushed to branch master at gajim / gajim Commits: 62b210a9 by Philipp Hörist at 2018-11-11T09:40:48Z Pass account to plugins when encrypting a file - - - - - 2 changed files: - gajim/common/modules/httpupload.py - gajim/gui_interface.py Changes: ===================================== gajim/common/modules/httpupload.py ===================================== @@ -167,7 +167,7 @@ class HTTPUpload: return if encryption is not None: - app.interface.encrypt_file(file, self._request_slot) + app.interface.encrypt_file(file, self._account, self._request_slot) else: self._request_slot(file) ===================================== gajim/gui_interface.py ===================================== @@ -1136,13 +1136,13 @@ class Interface: chat_control.session, groupchat) - def encrypt_file(self, file, callback): + def encrypt_file(self, file, account, callback): app.nec.push_incoming_event(HTTPUploadProgressEvent( None, status='encrypt', file=file)) encryption = file.encryption plugin = app.plugin_manager.encryption_plugins[encryption] if hasattr(plugin, 'encrypt_file'): - plugin.encrypt_file(file, None, callback) + plugin.encrypt_file(file, account, callback) else: app.nec.push_incoming_event(HTTPUploadProgressEvent( None, status='close', file=file)) View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/62b210a9e5db79413ca1eef90b74698d33… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/62b210a9e5db79413ca1eef90b74698d33… You're receiving this email because of your account on dev.gajim.org.

1 0

← Newer
1
2
3
4
5
6
7
8
9
Older →

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

Commits November 2018