AaronFiore/electrum
1
0
Fork 0
Code Activity

lnworker.add_peer: don't block event loop with DNS

Browse Source
This commit is contained in:
SomberNight
2020-03-03 21:04:54 +01:00
parent fbd0c9aae8
commit 01981f71fb
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
electrum/lnworker.py
View File

@@ -857,8 +857,11 @@ class LNWallet(LNWorker):
if not addrs: if not addrs:
raise ConnStringFormatError(_('Don\'t know any addresses for node:') + ' ' + bh2u(node_id)) raise ConnStringFormatError(_('Don\'t know any addresses for node:') + ' ' + bh2u(node_id))
host, port, timestamp = self.choose_preferred_address(addrs) host, port, timestamp = self.choose_preferred_address(addrs)
port = int(port)
# Try DNS-resolving the host (if needed). This is simply so that
# the caller gets a nice exception if it cannot be resolved.
try: try:
socket.getaddrinfo(host, int(port)) await asyncio.get_event_loop().getaddrinfo(host, port)
except socket.gaierror: except socket.gaierror:
raise ConnStringFormatError(_('Hostname does not resolve (getaddrinfo failed)')) raise ConnStringFormatError(_('Hostname does not resolve (getaddrinfo failed)'))
# add peer # add peer