fix(sqllab): default PostgreSQL port to 5432 in the dynamic connection form

This commit is contained in:
sadpandajoe
2026-08-20 00:40:45 +00:00
committed by Joe Li
parent faf7c34c0a
commit 8a8940557d
5 changed files with 398 additions and 12 deletions
@@ -491,6 +491,7 @@ def test_base_parameters_mixin():
"minimum": 0,
"maximum": 65536,
"description": "Database port",
"nullable": True,
},
"password": {"type": "string", "nullable": True, "description": "Password"},
"username": {"type": "string", "nullable": True, "description": "Username"},
@@ -504,7 +505,10 @@ def test_base_parameters_mixin():
"type": "boolean",
},
},
"required": ["database", "host", "port", "username"],
# ``port`` is intentionally not required: a blank port falls back to
# Postgres's own default (5432) in ``PostgresEngineSpec.build_sqlalchemy_uri``.
"required": ["database", "host", "username"],
"additionalProperties": False,
}