Skip to content

Commit 0907774

Browse files
committed
timeseries: Fixed codeql issues in tests
Signed-off-by: Pooja Kumbharkar <pooja.kumbharkar@intel.com>
1 parent 902b3c9 commit 0907774

File tree

2 files changed

+3
-9
lines changed

2 files changed

+3
-9
lines changed

manufacturing-ai-suite/industrial-edge-insights-time-series/tests/utils/docker_utils.py

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -643,7 +643,7 @@ def update_env_file(file_path=None, values=None):
643643
if not values["S3_STORAGE_USERNAME"]:
644644
logger.error("S3_STORAGE_USERNAME is empty in values dictionary")
645645
return False
646-
logger.info(f"Updating S3_STORAGE_USERNAME with value: {values['S3_STORAGE_USERNAME']}")
646+
logger.info(f"Updating S3_STORAGE_USERNAME with value: REDACTED for security")
647647

648648
if "S3_STORAGE_PASSWORD" in values:
649649
if not values["S3_STORAGE_PASSWORD"]:
@@ -2761,13 +2761,7 @@ def generate_multimodal_test_credentials(case_type="valid", invalid_field=None):
27612761
# Combine basic and multimodal credentials
27622762
basic_credentials.update(multimodal_vars)
27632763

2764-
# Log the generated credentials for debugging (without sensitive values)
2765-
logger.info(f"Generated multimodal credentials for case '{case_type}':")
2766-
for key in basic_credentials.keys():
2767-
if 'PASSWORD' in key or 'SECRET' in key:
2768-
logger.info(f" {key}: [REDACTED]")
2769-
else:
2770-
logger.info(f" {key}: {basic_credentials[key]}")
2764+
logger.info(f"Generated {len(basic_credentials)} multimodal credentials for case '{case_type}'")
27712765

27722766
return basic_credentials
27732767

manufacturing-ai-suite/industrial-edge-insights-time-series/tests/utils/helm_utils.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -690,7 +690,7 @@ def fetch_influxdb_credentials(chart_path):
690690
logger.info("Yaml file values: %s", values)
691691
# Print or return the values
692692
logger.info(f"INFLUXDB_USERNAME: {influxdb_username}")
693-
logger.info(f"INFLUXDB_PASSWORD: {influxdb_password}")
693+
logger.info(f"INFLUXDB_PASSWORD: REDACTED")
694694
logger.info(f"INFLUXDB_RETENTION_DURATION: {influxdb_retention_duration}")
695695
if not influxdb_username or not influxdb_password or not influxdb_retention_duration:
696696
logger.error("InfluxDB credentials not found in values.yaml.")

0 commit comments

Comments
 (0)