Skip to content

Commit

Permalink
analyzer: fix two crashes
Browse files Browse the repository at this point in the history
OSError from 'sss_analyze error list'

PermissionError from 'sss_analyze request list' run without sudo

Reviewed-by: Alejandro López <[email protected]>
Reviewed-by: Tomáš Halman <[email protected]>
(cherry picked from commit 0bb1364)
  • Loading branch information
justin-stephenson authored and alexey-tikhonov committed Nov 26, 2024
1 parent ba2b247 commit 9e3fbbc
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions src/tools/analyzer/source_files.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,9 +31,8 @@ def __iter__(self):
with open(files) as file:
for line in file:
yield line
except FileNotFoundError as err:
logger.warning("Could not find domain log file, skipping")
logger.warning(err)
except Exception as e:
logger.warning(e)
continue

def resolve_path(self, path):
Expand Down Expand Up @@ -70,7 +69,6 @@ def set_component(self, component, child):
elif component == self.Component.BE:
domains = self.get_domain_logfiles(child)
if not domains:
raise IOError
# error: No domains found?
return
for dom in domains:
self.log_files.append(dom)

0 comments on commit 9e3fbbc

Please sign in to comment.