Philipp Hörist pushed to branch master at gajim / gajim
Commits:
-
54929665
by Philipp Hörist at 2020-01-23T19:39:58+01:00
7 changed files:
- gajim/common/connection.py
- gajim/common/jingle_session.py
- gajim/common/jingle_transport.py
- gajim/common/jingle_xtls.py
- gajim/common/modules/adhoc_commands.py
- gajim/common/modules/mam.py
- gajim/common/zeroconf/client_zeroconf.py
Changes:
| ... | ... | @@ -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)
|
| ... | ... | @@ -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
|
| ... | ... | @@ -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'],
|
| ... | ... | @@ -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)
|
| ... | ... | @@ -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)
|
| ... | ... | @@ -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 |
|
| ... | ... | @@ -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
|