From d08ec56a73839c2fc64a4c38eaf523f513e17cff Mon Sep 17 00:00:00 2001 From: Sahil Yadav Date: Mon, 11 May 2026 19:36:24 +0530 Subject: [PATCH 1/2] fix: resolve publisher resource leak and field parse validation (#886) --- trustgraph-base/trustgraph/objects/field.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/trustgraph-base/trustgraph/objects/field.py b/trustgraph-base/trustgraph/objects/field.py index e4d84991..84db32aa 100644 --- a/trustgraph-base/trustgraph/objects/field.py +++ b/trustgraph-base/trustgraph/objects/field.py @@ -29,8 +29,8 @@ class Field: parts = defn.split(":") - if len(parts) == 0: - raise RuntimeError("Field definition cannot be empty") + if len(parts) > 5: + raise RuntimeError("Too many fields in definition") if len(parts) == 1: parts.append("string") if len(parts) == 2: parts.append("0") @@ -43,7 +43,7 @@ class Field: try: type = FieldType[type.upper()] - except: + except KeyError: raise RuntimeError(f"Field type {type} is not known") pri = True if pri == "pri" else False From dd974b0caca9af6bc0a18eb2bac2ef332f12916b Mon Sep 17 00:00:00 2001 From: KOTHA-SRIVIBHU Date: Mon, 11 May 2026 19:41:30 +0530 Subject: [PATCH 2/2] fix: replace bare excepts in NLTK initialization (#896) --- .../extract/kg/ontology/text_processor.py | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/trustgraph-flow/trustgraph/extract/kg/ontology/text_processor.py b/trustgraph-flow/trustgraph/extract/kg/ontology/text_processor.py index 685699d1..4e0d79a5 100644 --- a/trustgraph-flow/trustgraph/extract/kg/ontology/text_processor.py +++ b/trustgraph-flow/trustgraph/extract/kg/ontology/text_processor.py @@ -18,24 +18,26 @@ try: except LookupError: try: nltk.download('punkt_tab', quiet=True) - except: + except Exception as e: + logger.warning(f"Failed to download punkt_tab: {e}. Attempting fallback to punkt.") # Fallback to older punkt if punkt_tab not available try: nltk.download('punkt', quiet=True) - except: - pass + except Exception as e: + logger.error(f"Failed to download fallback punkt: {e}. NLTK data is missing.") try: nltk.data.find('taggers/averaged_perceptron_tagger_eng') except LookupError: try: nltk.download('averaged_perceptron_tagger_eng', quiet=True) - except: + except Exception as e: + logger.warning(f"Failed to download averaged_perceptron_tagger_eng: {e}. Attempting fallback.") # Fallback to older name try: nltk.download('averaged_perceptron_tagger', quiet=True) - except: - pass + except Exception as e: + logger.error(f"Failed to download fallback averaged_perceptron_tagger: {e}. NLTK data is missing.") try: nltk.data.find('corpora/stopwords')