Reduce duplicate code
Clean up DNS interface logic
This commit is contained in:
+6
-9
@@ -95,16 +95,13 @@ async def set_dns_servers(client: SagemcomClient, dns_servers: tuple[IPv4Address
|
|||||||
for i, dns_server in enumerate(dns_servers, start=1):
|
for i, dns_server in enumerate(dns_servers, start=1):
|
||||||
try:
|
try:
|
||||||
await client.set_value_by_xpath(f"Device/DNS/Relay/Forwardings/Forwarding[@uid={i}]/DNSServer", dns_server)
|
await client.set_value_by_xpath(f"Device/DNS/Relay/Forwardings/Forwarding[@uid={i}]/DNSServer", dns_server)
|
||||||
|
iface = 'Device/IP/Interfaces/Interface[IP_DATA]'
|
||||||
if dns_server.is_private:
|
if dns_server.is_private:
|
||||||
await client.set_value_by_xpath(
|
iface = 'Device/IP/Interfaces/Interface[IP_BR_LAN]
|
||||||
f"Device/DNS/Relay/Forwardings/Forwarding[@uid={i}]/Interface",
|
await client.set_value_by_xpath(
|
||||||
'Device/IP/Interfaces/Interface[IP_BR_LAN]'
|
f"Device/DNS/Relay/Forwardings/Forwarding[@uid={i}]/Interface",
|
||||||
)
|
iface
|
||||||
elif dns_server.is_global:
|
)
|
||||||
await client.set_value_by_xpath(
|
|
||||||
f"Device/DNS/Relay/Forwardings/Forwarding[@uid={i}]/Interface",
|
|
||||||
'Device/IP/Interfaces/Interface[IP_DATA]'
|
|
||||||
)
|
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
click.echo(e, err=True)
|
click.echo(e, err=True)
|
||||||
break
|
break
|
||||||
|
|||||||
Reference in New Issue
Block a user