fix: change ssl_mode to sslmode (#408)

* fix: change ssl_mode to sslmode * fix: remove comma causing sslkey=(None,) when MF_METADATA_DB_SSL_KEY_PATH not defined
Netflix · Jan 18, 2024 · da31cdc · da31cdc
1 parent b5b61d1
commit da31cdc
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/run_goose.py b/run_goose.py
@@ -61,7 +61,7 @@ def main():
     ssl_root_cert_path = os.environ.get("MF_METADATA_DB_SSL_ROOT_CERT")
 
     if ssl_mode in ['allow', 'prefer', 'require', 'verify-ca', 'verify-full']:
-        ssl_query = f'ssl_mode={ssl_mode}'
+        ssl_query = f'sslmode={ssl_mode}'
         if ssl_cert_path is not None:
             ssl_query = f'{ssl_query}&sslcert={ssl_cert_path}'
         if ssl_key_path is not None:

diff --git a/services/utils/__init__.py b/services/utils/__init__.py
@@ -228,7 +228,7 @@ def __init__(self,
         self._database_name = os.environ.get(prefix + "NAME", database_name)
         self._ssl_mode = os.environ.get(prefix + "SSL_MODE", ssl_mode)
         self._ssl_cert_path = os.environ.get(prefix + "SSL_CERT_PATH", ssl_cert_path)
-        self._ssl_key_path = os.environ.get(prefix + "SSL_KEY_PATH", ssl_key_path),
+        self._ssl_key_path = os.environ.get(prefix + "SSL_KEY_PATH", ssl_key_path)
         self._ssl_root_cert_path = os.environ.get(prefix + "SSL_ROOT_CERT_PATH", ssl_root_cert_path)
         conn_str_required_values = [
             self._host,