azalea/sagemcom
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Update xmo-cli.py

Browse Source 
Use a set rather than overwriting a list items
This commit is contained in:
up-n-atom
2023-09-11 19:44:54 -04:00
committed by GitHub
parent ab770e402a
commit 4985426ea0
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
xmo-cli.py
+4 -4
View File
@@ -76,13 +76,13 @@ async def get_dns(ctx: click.Context) -> None:
await ctx.invoke(get_value, path='Device/DNS') await ctx.invoke(get_value, path='Device/DNS')
def validate_dns_servers(ctx, param, value): def validate_dns_servers(ctx, param, value) -> tuple[IPv4Address]:
if isinstance(value, tuple): if isinstance(value, tuple):
return value return value
dns_servers = value.split(' ', maxsplit=1) dns_servers = set()
for i, dns_server in enumerate(dns_servers): for dns_server in value.split(' ', maxsplit=1):
try: try:
dns_servers[i] = IPv4Address(dns_server) dns_servers.add(IPv4Address(dns_server))
except: except:
raise click.BadParameter(f"Invalid IPv4 address: {dns_server}") raise click.BadParameter(f"Invalid IPv4 address: {dns_server}")
return tuple(dns_servers) return tuple(dns_servers)