Philipp Hörist pushed to branch master at gajim / gajim

Commits:

7 changed files:

Changes:

  • gajim/common/connection.py
    ... ... @@ -48,6 +48,7 @@ import OpenSSL.crypto
    48 48
     import nbxmpp
    
    49 49
     from nbxmpp.const import Realm
    
    50 50
     from nbxmpp.const import Event
    
    51
    +from nbxmpp.util import generate_id
    
    51 52
     
    
    52 53
     from gajim import common
    
    53 54
     from gajim.common import helpers
    
    ... ... @@ -232,7 +233,7 @@ class CommonConnection:
    232 233
                               namespace=nbxmpp.NS_CORRECT)
    
    233 234
     
    
    234 235
             # XEP-0359
    
    235
    -        obj.stanza_id = self.connection.getAnID()
    
    236
    +        obj.stanza_id = generate_id()
    
    236 237
             msg_iq.setID(obj.stanza_id)
    
    237 238
             if obj.message:
    
    238 239
                 msg_iq.setOriginID(obj.stanza_id)
    
    ... ... @@ -1625,7 +1626,7 @@ class Connection(CommonConnection, ConnectionHandlers):
    1625 1626
                                     typ='groupchat',
    
    1626 1627
                                     xhtml=xhtml)
    
    1627 1628
     
    
    1628
    -        obj.stanza_id = self.connection.getAnID()
    
    1629
    +        obj.stanza_id = generate_id()
    
    1629 1630
             msg_iq.setID(obj.stanza_id)
    
    1630 1631
             if obj.message:
    
    1631 1632
                 msg_iq.setOriginID(obj.stanza_id)
    

  • gajim/common/jingle_session.py
    ... ... @@ -31,6 +31,7 @@ import logging
    31 31
     from enum import Enum, unique
    
    32 32
     
    
    33 33
     import nbxmpp
    
    34
    +from nbxmpp.util import generate_id
    
    34 35
     
    
    35 36
     from gajim.common import app
    
    36 37
     from gajim.common.jingle_transport import get_jingle_transport
    
    ... ... @@ -103,7 +104,7 @@ class JingleSession:
    103 104
             # what state is session in? (one from JingleStates)
    
    104 105
             self.state = JingleStates.ENDED
    
    105 106
             if not sid:
    
    106
    -            sid = con.connection.getAnID()
    
    107
    +            sid = generate_id()
    
    107 108
             self.sid = sid # sessionid
    
    108 109
             # iq stanza id, used to determine which sessions to summon callback
    
    109 110
             # later on when iq-result stanza arrives
    

  • gajim/common/jingle_transport.py
    ... ... @@ -24,6 +24,7 @@ import socket
    24 24
     from enum import IntEnum, unique
    
    25 25
     
    
    26 26
     import nbxmpp
    
    27
    +from nbxmpp.util import generate_id
    
    27 28
     
    
    28 29
     from gajim.common import app
    
    29 30
     
    
    ... ... @@ -96,7 +97,7 @@ class JingleTransport:
    96 97
         def set_connection(self, conn):
    
    97 98
             self.connection = conn
    
    98 99
             if not self.sid:
    
    99
    -            self.sid = self.connection.connection.getAnID()
    
    100
    +            self.sid = generate_id()
    
    100 101
     
    
    101 102
         def set_file_props(self, file_props):
    
    102 103
             self.file_props = file_props
    
    ... ... @@ -198,7 +199,7 @@ class JingleTransportSocks5(JingleTransport):
    198 199
     
    
    199 200
             candidate = {
    
    200 201
                 'host': self.connection.peerhost[0],
    
    201
    -            'candidate_id': self.connection.connection.getAnID(),
    
    202
    +            'candidate_id': generate_id(),
    
    202 203
                 'port': port,
    
    203 204
                 'type': 'direct',
    
    204 205
                 'jid': self.ourjid,
    
    ... ... @@ -214,7 +215,7 @@ class JingleTransportSocks5(JingleTransport):
    214 215
                     addr != '::1':
    
    215 216
                         candidate = {
    
    216 217
                             'host': addr,
    
    217
    -                        'candidate_id': self.connection.connection.getAnID(),
    
    218
    +                        'candidate_id': generate_id(),
    
    218 219
                             'port': port,
    
    219 220
                             'type': 'direct',
    
    220 221
                             'jid': self.ourjid,
    
    ... ... @@ -238,7 +239,7 @@ class JingleTransportSocks5(JingleTransport):
    238 239
                                 continue
    
    239 240
                             candidate = {
    
    240 241
                                 'host': addr,
    
    241
    -                            'candidate_id': self.connection.connection.getAnID(),
    
    242
    +                            'candidate_id': generate_id(),
    
    242 243
                                 'port': port,
    
    243 244
                                 'type': 'direct',
    
    244 245
                                 'jid': self.ourjid,
    
    ... ... @@ -256,7 +257,7 @@ class JingleTransportSocks5(JingleTransport):
    256 257
                                 continue
    
    257 258
                             candidate = {
    
    258 259
                                 'host': addr,
    
    259
    -                            'candidate_id': self.connection.connection.getAnID(),
    
    260
    +                            'candidate_id': generate_id(),
    
    260 261
                                 'port': port,
    
    261 262
                                 'type': 'direct',
    
    262 263
                                 'jid': self.ourjid,
    
    ... ... @@ -286,7 +287,7 @@ class JingleTransportSocks5(JingleTransport):
    286 287
                 for host in hosts:
    
    287 288
                     candidate = {
    
    288 289
                         'host': host,
    
    289
    -                    'candidate_id': self.connection.connection.getAnID(),
    
    290
    +                    'candidate_id': generate_id(),
    
    290 291
                         'port': port,
    
    291 292
                         'type': 'direct',
    
    292 293
                         'jid': self.ourjid,
    
    ... ... @@ -313,7 +314,7 @@ class JingleTransportSocks5(JingleTransport):
    313 314
                 for proxyhost in proxyhosts:
    
    314 315
                     candidate = {
    
    315 316
                         'host': proxyhost['host'],
    
    316
    -                    'candidate_id': self.connection.connection.getAnID(),
    
    317
    +                    'candidate_id': generate_id(),
    
    317 318
                         'port': int(proxyhost['port']),
    
    318 319
                         'type': 'proxy',
    
    319 320
                         'jid': proxyhost['jid'],
    

  • gajim/common/jingle_xtls.py
    ... ... @@ -18,6 +18,7 @@ import os
    18 18
     from OpenSSL import SSL, crypto
    
    19 19
     
    
    20 20
     import nbxmpp
    
    21
    +from nbxmpp.util import generate_id
    
    21 22
     from gajim.common import app
    
    22 23
     from gajim.common import configpaths
    
    23 24
     
    
    ... ... @@ -188,7 +189,7 @@ def check_cert(jid, fingerprint):
    188 189
     
    
    189 190
     def send_cert_request(con, to_jid):
    
    190 191
         iq = nbxmpp.Iq('get', to=to_jid)
    
    191
    -    id_ = con.connection.getAnID()
    
    192
    +    id_ = generate_id()
    
    192 193
         iq.setAttr('id', id_)
    
    193 194
         pubkey = iq.setTag('pubkeys')
    
    194 195
         pubkey.setNamespace(nbxmpp.NS_PUBKEY_PUBKEY)
    

  • gajim/common/modules/adhoc_commands.py
    ... ... @@ -22,6 +22,7 @@
    22 22
     import nbxmpp
    
    23 23
     from nbxmpp.structs import StanzaHandler
    
    24 24
     from nbxmpp.modules import dataforms
    
    25
    +from nbxmpp.util import generate_id
    
    25 26
     
    
    26 27
     from gajim.common import app
    
    27 28
     from gajim.common import helpers
    
    ... ... @@ -314,7 +315,7 @@ class AdHocCommands(BaseModule):
    314 315
                     raise nbxmpp.NodeProcessed
    
    315 316
     
    
    316 317
                 # generate new sessionid
    
    317
    -            sessionid = self._con.connection.getAnID()
    
    318
    +            sessionid = generate_id()
    
    318 319
     
    
    319 320
                 # create new instance and run it
    
    320 321
                 obj = newcmd(conn=self, jid=jid, sessionid=sessionid)
    

  • gajim/common/modules/mam.py
    ... ... @@ -18,6 +18,7 @@ import time
    18 18
     from datetime import datetime, timedelta
    
    19 19
     
    
    20 20
     import nbxmpp
    
    21
    +from nbxmpp.util import generate_id
    
    21 22
     from nbxmpp.structs import StanzaHandler
    
    22 23
     
    
    23 24
     from gajim.common import app
    
    ... ... @@ -256,7 +257,7 @@ class MAM(BaseModule):
    256 257
             return valid_id == properties.mam.query_id
    
    257 258
     
    
    258 259
         def _get_query_id(self, jid):
    
    259
    -        query_id = self._con.connection.getAnID()
    
    260
    +        query_id = generate_id()
    
    260 261
             self._mam_query_ids[jid] = query_id
    
    261 262
             return query_id
    
    262 263
     
    

  • gajim/common/zeroconf/client_zeroconf.py
    ... ... @@ -34,6 +34,7 @@ from nbxmpp.idlequeue import IdleObject
    34 34
     from nbxmpp.transports import DATA_RECEIVED
    
    35 35
     from nbxmpp.transports import DATA_SENT
    
    36 36
     from nbxmpp.transports import DATA_ERROR
    
    37
    +from nbxmpp.util import generate_id
    
    37 38
     
    
    38 39
     from gajim.common import app
    
    39 40
     from gajim.common.zeroconf import zeroconf
    
    ... ... @@ -184,7 +185,7 @@ class P2PClient(IdleObject):
    184 185
                     thread_id = stanza.getThread()
    
    185 186
                     id_ = stanza.getID()
    
    186 187
                     if not id_:
    
    187
    -                    id_ = self.Dispatcher.getAnID()
    
    188
    +                    id_ = generate_id()
    
    188 189
                     if self.fd in self.conn_holder.ids_of_awaiting_messages:
    
    189 190
                         self.conn_holder.ids_of_awaiting_messages[self.fd].append((
    
    190 191
                             id_, thread_id))
    
    ... ... @@ -209,7 +210,7 @@ class P2PClient(IdleObject):
    209 210
                 thread_id = stanza.getThread()
    
    210 211
                 id_ = stanza.getID()
    
    211 212
                 if not id_:
    
    212
    -                id_ = self.Dispatcher.getAnID()
    
    213
    +                id_ = generate_id()
    
    213 214
                 if self.fd in self.conn_holder.ids_of_awaiting_messages:
    
    214 215
                     self.conn_holder.ids_of_awaiting_messages[self.fd].append((id_,
    
    215 216
                         thread_id))
    
    ... ... @@ -800,12 +801,6 @@ class ClientZeroconf:
    800 801
             P2PClient(None, addresses_, self,
    
    801 802
                 [(stanza, is_message)], to, on_ok=on_ok, on_not_ok=on_not_ok)
    
    802 803
     
    
    803
    -    def getAnID(self):
    
    804
    -        """
    
    805
    -        Generate a random id
    
    806
    -        """
    
    807
    -        return ''.join(Random().sample(string.ascii_letters + string.digits, 6))
    
    808
    -
    
    809 804
         def RegisterDisconnectHandler(self, handler):
    
    810 805
             """
    
    811 806
             Register handler that will be called on disconnect