From 5820bc95c1c91bc3711ebf0ad177c3af35cd6b24 Mon Sep 17 00:00:00 2001 From: Janik Besendorf Date: Wed, 25 Mar 2026 08:44:16 +0100 Subject: [PATCH] Replace bare KeyError catch with explicit key check in net_base --- src/mvt/ios/modules/net_base.py | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/src/mvt/ios/modules/net_base.py b/src/mvt/ios/modules/net_base.py index 1773e29..e0952ed 100644 --- a/src/mvt/ios/modules/net_base.py +++ b/src/mvt/ios/modules/net_base.py @@ -311,14 +311,11 @@ class NetBase(IOSExtraction): self.results = sorted(self.results, key=operator.itemgetter("first_isodate")) def check_indicators(self) -> None: - # Check for manipulated process records. - # TODO: Catching KeyError for live_isodate for retro-compatibility. - # This is not very good. - try: + # check_manipulated/find_deleted require "live_isodate" and + # "live_proc_id" keys which may be absent in older result formats. + if self.results and "live_isodate" in self.results[0]: self.check_manipulated() self.find_deleted() - except KeyError: - pass if not self.indicators: return