Skip to content

Commit

Permalink
T6587: Raise an error when trying to get information about network in…
Browse files Browse the repository at this point in the history
…terfaces that don't exist
  • Loading branch information
HollyGurza committed Jul 22, 2024
1 parent 9c64df5 commit 3ae1177
Showing 1 changed file with 11 additions and 2 deletions.
13 changes: 11 additions & 2 deletions src/op_mode/interfaces.py
Original file line number Diff line number Diff line change
Expand Up @@ -450,7 +450,13 @@ def show(raw: bool, intf_name: typing.Optional[str],
vif: bool, vrrp: bool):
data = _get_raw_data(intf_name, intf_type, vif, vrrp)
if raw:
return data
if intf_name is not None and len(data) <= 1:
try:
return data[0]
except IndexError:
raise vyos.opmode.UnconfiguredObject(f"Interface {intf_name} does not exist")
else:
return data
return _format_show_data(data)

def show_summary(raw: bool, intf_name: typing.Optional[str],
Expand All @@ -459,7 +465,10 @@ def show_summary(raw: bool, intf_name: typing.Optional[str],
data = _get_summary_data(intf_name, intf_type, vif, vrrp)
if raw:
if intf_name is not None and len(data) <= 1:
return data[0] if data else {}
try:
return data[0]
except IndexError:
raise vyos.opmode.UnconfiguredObject(f"Interface {intf_name} does not exist")
else:
return data
return _format_show_summary(data)
Expand Down

0 comments on commit 3ae1177

Please sign in to comment.