legacy: use public_url, url is not very useful

also refactor vars so it is easier to read
This commit is contained in:
pedro 2024-11-12 01:15:34 +01:00
parent b4d86fcc12
commit 4cb3e34b6b
1 changed files with 7 additions and 5 deletions

View File

@ -319,13 +319,15 @@ def send_snapshot_to_devicehub(snapshot, token, url, legacy):
if legacy: if legacy:
try: try:
response = json.loads(response_text) response = json.loads(response_text)
if response.get('url'): public_url = response.get('public_url')
dhid = response.get('dhid')
if public_url:
# apt install qrencode # apt install qrencode
qr = "echo {} | qrencode -t ANSI".format(response['url']) qr = "echo {} | qrencode -t ANSI".format(public_url)
print(exec_cmd(qr)) print(exec_cmd(qr))
print("url: {}".format(response['url'])) print("url: {}".format(public_url))
if response.get("dhid"): if dhid:
print("dhid: {}".format(response['dhid'])) print("dhid: {}".format(dhid))
except Exception: except Exception:
logger.error(response_text) logger.error(response_text)
else: else: