jshiffer/slixmpp
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

basexmpp: make_iq no longer defaults to id="0"

Browse Source 
having a hardcoded default ID in make_iq is a bad idea, particularly
since it will overwrite the (good) id produced byt Iq() when a stream is
available.

This is arguably a breaking change, but I certainly hope it is not
breaking anything in the real world.
This commit is contained in:
mathieui 2025-01-26 18:03:15 +01:00
parent 2e736bc715
commit 1482bcc395
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
slixmpp/basexmpp.py
View File

@ -315,13 +315,12 @@ class BaseXMPP(XMLStream):
pres['lang'] = self.default_lang pres['lang'] = self.default_lang
return pres return pres
def make_iq(self, id: str = "0", ifrom: OptJidStr = None, def make_iq(self, id: Optional[str] = None, ifrom: OptJidStr = None,
ito: OptJidStr = None, itype: Optional[IqTypes] = None, ito: OptJidStr = None, itype: Optional[IqTypes] = None,
iquery: Optional[str] = None) -> stanza.Iq: iquery: Optional[str] = None) -> stanza.Iq:
"""Create a new :class:`~.Iq` stanza with a given Id and from JID. """Create a new :class:`~.Iq` stanza with a given Id and from JID.
:param id: An ideally unique ID value for this stanza thread. :param id: An ideally unique ID value for this stanza thread.
Defaults to 0.
:param ifrom: The from :class:`~.JID` :param ifrom: The from :class:`~.JID`
to use for this stanza. to use for this stanza.
:param ito: The destination :class:`~.JID` :param ito: The destination :class:`~.JID`
@ -332,6 +331,7 @@ class BaseXMPP(XMLStream):
:param iquery: Optional namespace for adding a query element. :param iquery: Optional namespace for adding a query element.
""" """
iq = self.Iq() iq = self.Iq()
if id is not None:
iq['id'] = str(id) iq['id'] = str(id)
iq['to'] = ito iq['to'] = ito
iq['from'] = ifrom iq['from'] = ifrom