Clean logs for C002. (#14)

Decathlon · Nov 15, 2024 · aec0e25 · aec0e25
1 parent 032d5d1
commit aec0e25
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 185 deletions.
diff --git a/dblinter/default_config.yaml b/dblinter/default_config.yaml
@@ -11,7 +11,7 @@ cluster:
         - downsize max_connections or upsize memory.
   - name: PgHbaEntriesWithMethodTrustOrPasswordShouldNotExists
     ruleid: C002
-    enabled: false
+    enabled: True
     params:
       - warning: 1
     context:

diff --git a/dblinter/rules/C002/PgHbaEntriesWithMethodTrustOrPasswordShouldNotExists.py b/dblinter/rules/C002/PgHbaEntriesWithMethodTrustOrPasswordShouldNotExists.py
@@ -14,12 +14,10 @@ def pg_hba_entries_with_trust_or_password_method(
         "SELECT count(*) FROM pg_catalog.pg_hba_file_rules WHERE auth_method in ('trust','password')"
     )[0][0]
 
-    warning = extract_param(param, "warning")
+    warning = int(extract_param(param, "warning"))
 
     if trust_password >= warning:
         message_args = (trust_password, warning)
         sarif_document.add_check(
             self.get_ruleid_from_function_name(), message_args, db.database, context
         )
-    else:
-        LOGGER.error("Error: param warning is missing in the configuration file")
diff --git a/default_config.yaml b/default_config.yaml