Connections using `connectString` store redundant `port` property

[gjsjohnmurray]

Describe the bug After defining a connection that uses a connection string (connectString property) the JSON includes a redundant port property.

To Reproduce Steps to reproduce the behavior:

1. Start defining a new connection that uses the MSSQL, MySQL or PostgreSQL driver.
2. Set connection method to "Connection String".
3. Save the definition and observe that the JSON includes a redundant port property with a default value.

Expected behavior No port property in the JSON of a connection definition that has a connectString property.

Desktop (please complete the following information):

• SQLTools Version: 0.27.0
• VS Code Version: 1.74.2
• OS: Windows
• Driver:
  • [x] PostgreSQL/Redshift 0.5.0
  • [x] MySQL/MariaDB 0.5.0
  • [x] MSSQL/Azure 0.4.0
  • [ ] SQLite
  • [ ] Other? Which...
• Database version: n/a