Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
python/qmp/legacy: allow using sockets for connect()
Instead of asserting that we have an address, allow the use of sockets
instead of addresses during a call to connect().

Signed-off-by: John Snow <jsnow@redhat.com>
Message-id: 20230517163406.2593480-3-jsnow@redhat.com
Signed-off-by: John Snow <jsnow@redhat.com>
  • Loading branch information
jnsnow committed May 31, 2023
1 parent 9341b2a commit b8d4ca1
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions python/qemu/qmp/legacy.py
Expand Up @@ -150,12 +150,13 @@ def connect(self, negotiate: bool = True) -> Optional[QMPMessage]:
:return: QMP greeting dict, or None if negotiate is false
:raise ConnectError: on connection errors
"""
assert self._address is not None
addr_or_sock = self._address or self._sock
assert addr_or_sock is not None
self._qmp.await_greeting = negotiate
self._qmp.negotiate = negotiate

self._sync(
self._qmp.connect(self._address)
self._qmp.connect(addr_or_sock)
)
return self._get_greeting()

Expand Down

0 comments on commit b8d4ca1

Please sign in to comment.