[CI] Format code with Black

This commit is contained in:
yunohost-bot 2023-01-30 17:34:24 +00:00
parent 82d30f02e2
commit 2d024557a5
4 changed files with 44 additions and 12 deletions

View file

@ -125,7 +125,11 @@ class MyDiagnoser(Diagnoser):
yield dict( yield dict(
meta={"test": "ipv4"}, meta={"test": "ipv4"},
data={"global": ipv4, "local": get_local_ip("ipv4")}, data={"global": ipv4, "local": get_local_ip("ipv4")},
status="SUCCESS" if ipv4 else "ERROR" if is_ipvx_important(4) else "WARNING", status="SUCCESS"
if ipv4
else "ERROR"
if is_ipvx_important(4)
else "WARNING",
summary="diagnosis_ip_connected_ipv4" if ipv4 else "diagnosis_ip_no_ipv4", summary="diagnosis_ip_connected_ipv4" if ipv4 else "diagnosis_ip_no_ipv4",
details=["diagnosis_ip_global", "diagnosis_ip_local"] if ipv4 else None, details=["diagnosis_ip_global", "diagnosis_ip_local"] if ipv4 else None,
) )
@ -133,11 +137,19 @@ class MyDiagnoser(Diagnoser):
yield dict( yield dict(
meta={"test": "ipv6"}, meta={"test": "ipv6"},
data={"global": ipv6, "local": get_local_ip("ipv6")}, data={"global": ipv6, "local": get_local_ip("ipv6")},
status="SUCCESS" if ipv6 else "ERROR" if is_ipvx_important(6) else "WARNING", status="SUCCESS"
if ipv6
else "ERROR"
if is_ipvx_important(6)
else "WARNING",
summary="diagnosis_ip_connected_ipv6" if ipv6 else "diagnosis_ip_no_ipv6", summary="diagnosis_ip_connected_ipv6" if ipv6 else "diagnosis_ip_no_ipv6",
details=["diagnosis_ip_global", "diagnosis_ip_local"] details=["diagnosis_ip_global", "diagnosis_ip_local"]
if ipv6 if ipv6
else ["diagnosis_ip_no_ipv6_tip_important" if is_ipvx_important(6) else "diagnosis_ip_no_ipv6_tip"], else [
"diagnosis_ip_no_ipv6_tip_important"
if is_ipvx_important(6)
else "diagnosis_ip_no_ipv6_tip"
],
) )
# TODO / FIXME : add some attempt to detect ISP (using whois ?) ? # TODO / FIXME : add some attempt to detect ISP (using whois ?) ?

View file

@ -47,7 +47,10 @@ class MyDiagnoser(Diagnoser):
ipversions = [] ipversions = []
ipv4 = Diagnoser.get_cached_report("ip", item={"test": "ipv4"}) or {} ipv4 = Diagnoser.get_cached_report("ip", item={"test": "ipv4"}) or {}
if ipv4.get("status") == "SUCCESS" or settings_get("misc.network.dns_exposure") != "ipv6": if (
ipv4.get("status") == "SUCCESS"
or settings_get("misc.network.dns_exposure") != "ipv6"
):
ipversions.append(4) ipversions.append(4)
# To be discussed: we could also make this check dependent on the # To be discussed: we could also make this check dependent on the
@ -121,7 +124,10 @@ class MyDiagnoser(Diagnoser):
for record in dnsrecords.get("items", []) for record in dnsrecords.get("items", [])
) )
if (failed == 4 and settings_get("misc.network.dns_exposure") in ["both", "ipv4"]) or (failed == 6 and ipv6_is_important()): if (
failed == 4
and settings_get("misc.network.dns_exposure") in ["both", "ipv4"]
) or (failed == 6 and ipv6_is_important()):
yield dict( yield dict(
meta={"port": port}, meta={"port": port},
data={ data={

View file

@ -77,7 +77,9 @@ class MyDiagnoser(Diagnoser):
ipversions = [] ipversions = []
ipv4 = Diagnoser.get_cached_report("ip", item={"test": "ipv4"}) or {} ipv4 = Diagnoser.get_cached_report("ip", item={"test": "ipv4"}) or {}
if ipv4.get("status") == "SUCCESS" and settings_get("misc.network.dns_exposure") in ["both", "ipv4"]: if ipv4.get("status") == "SUCCESS" and settings_get(
"misc.network.dns_exposure"
) in ["both", "ipv4"]:
ipversions.append(4) ipversions.append(4)
# To be discussed: we could also make this check dependent on the # To be discussed: we could also make this check dependent on the
@ -97,7 +99,10 @@ class MyDiagnoser(Diagnoser):
# "curl --head the.global.ip" will simply timeout... # "curl --head the.global.ip" will simply timeout...
if self.do_hairpinning_test: if self.do_hairpinning_test:
global_ipv4 = ipv4.get("data", {}).get("global", None) global_ipv4 = ipv4.get("data", {}).get("global", None)
if global_ipv4 and settings_get("misc.network.dns_exposure") in ["both", "ipv4"]: if global_ipv4 and settings_get("misc.network.dns_exposure") in [
"both",
"ipv4",
]:
try: try:
requests.head("http://" + global_ipv4, timeout=5) requests.head("http://" + global_ipv4, timeout=5)
except requests.exceptions.Timeout: except requests.exceptions.Timeout:
@ -148,7 +153,10 @@ class MyDiagnoser(Diagnoser):
if all( if all(
results[ipversion][domain]["status"] == "ok" for ipversion in ipversions results[ipversion][domain]["status"] == "ok" for ipversion in ipversions
): ):
if 4 in ipversions and settings_get("misc.network.dns_exposure") in ["both", "ipv4"]: if 4 in ipversions and settings_get("misc.network.dns_exposure") in [
"both",
"ipv4",
]:
self.do_hairpinning_test = True self.do_hairpinning_test = True
yield dict( yield dict(
meta={"domain": domain}, meta={"domain": domain},
@ -186,7 +194,9 @@ class MyDiagnoser(Diagnoser):
) )
AAAA_status = dnsrecords.get("data", {}).get("AAAA:@") AAAA_status = dnsrecords.get("data", {}).get("AAAA:@")
return AAAA_status in ["OK", "WRONG"] or settings_get("misc.network.dns_exposure") in ["both", "ipv6"] return AAAA_status in ["OK", "WRONG"] or settings_get(
"misc.network.dns_exposure"
) in ["both", "ipv6"]
if failed == 4 or ipv6_is_important_for_this_domain(): if failed == 4 or ipv6_is_important_for_this_domain():
yield dict( yield dict(

View file

@ -302,13 +302,17 @@ class MyDiagnoser(Diagnoser):
outgoing_ipversions = [] outgoing_ipversions = []
outgoing_ips = [] outgoing_ips = []
ipv4 = Diagnoser.get_cached_report("ip", {"test": "ipv4"}) or {} ipv4 = Diagnoser.get_cached_report("ip", {"test": "ipv4"}) or {}
if ipv4.get("status") == "SUCCESS" and settings_get("misc.network.dns_exposure") in ["both", "ipv4"]: if ipv4.get("status") == "SUCCESS" and settings_get(
"misc.network.dns_exposure"
) in ["both", "ipv4"]:
outgoing_ipversions.append(4) outgoing_ipversions.append(4)
global_ipv4 = ipv4.get("data", {}).get("global", {}) global_ipv4 = ipv4.get("data", {}).get("global", {})
if global_ipv4: if global_ipv4:
outgoing_ips.append(global_ipv4) outgoing_ips.append(global_ipv4)
if settings_get("email.smtp.smtp_allow_ipv6") or settings_get("misc.network.dns_exposure") in ["both", "ipv6"]: if settings_get("email.smtp.smtp_allow_ipv6") or settings_get(
"misc.network.dns_exposure"
) in ["both", "ipv6"]:
ipv6 = Diagnoser.get_cached_report("ip", {"test": "ipv6"}) or {} ipv6 = Diagnoser.get_cached_report("ip", {"test": "ipv6"}) or {}
if ipv6.get("status") == "SUCCESS": if ipv6.get("status") == "SUCCESS":
outgoing_ipversions.append(6) outgoing_ipversions.append(6)