From bbe92e2af365c93e113eb62b5e1337646c1f59f8 Mon Sep 17 00:00:00 2001 From: Onkar Ravgan Date: Thu, 22 Aug 2024 17:34:34 +0530 Subject: [PATCH] MINOR: Fix none password issue for mysql and postgresql databases (#17548) * fix none password issue * added warning --- ingestion/src/metadata/ingestion/connections/builders.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/ingestion/src/metadata/ingestion/connections/builders.py b/ingestion/src/metadata/ingestion/connections/builders.py index 5142f46ff5d..0994d9cd9fa 100644 --- a/ingestion/src/metadata/ingestion/connections/builders.py +++ b/ingestion/src/metadata/ingestion/connections/builders.py @@ -33,6 +33,9 @@ from metadata.generated.schema.entity.services.connections.database.common.iamAu from metadata.ingestion.connections.headers import inject_query_header_by_conn from metadata.ingestion.connections.secrets import connection_with_options_secrets from metadata.utils.constants import BUILDER_PASSWORD_ATTR +from metadata.utils.logger import cli_logger + +logger = cli_logger() @connection_with_options_secrets @@ -152,6 +155,9 @@ def _add_password(url: str, connection) -> str: Region=connection.authType.awsConfig.awsRegion, ) ) + if not password: + logger.warning("No password has been provided in connection") + password = SecretStr("") url += f":{quote_plus(password.get_secret_value())}" return url