particle-os/debian-forge
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

jsoncomm: remove destination from send

Browse source 
Now that jsoncomm.Socket is using a connection-oriented socket,
the destination in `socket.sendmsg` is ignored and thus can and
should be dropped from the `jsoncomm.Socket.send` method.
Adjust the tests accordingly.
This commit is contained in:
Christian Kellner 2020-07-28 12:58:57 +02:00 • committed by Tom Gundersen
parent 43db8082bd
commit 785f843901
2 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

4
osbuild/util/jsoncomm.py
View file

@ -321,7 +321,7 @@ class Socket(contextlib.AbstractContextManager):
return (payload, fdset, msg[3])
def send(self, payload: object, *, destination: Optional[str] = None, fds: Optional[list] = None):
def send(self, payload: object, *, fds: Optional[list] = None):
"""Send Message
Send a new message via this socket. This operation is synchronous. The
@ -352,5 +352,5 @@ class Socket(contextlib.AbstractContextManager):
if fds:
cmsg.append((socket.SOL_SOCKET, socket.SCM_RIGHTS, array.array("i", fds)))
n = self._socket.sendmsg([serialized], cmsg, 0, destination)
n = self._socket.sendmsg([serialized], cmsg, 0)
assert n == len(serialized)