These changes were originally made to fix #47206. The fix for #47206 was preempted by https://github.com/gravitational/teleport/pull/48811 fixing a duplicate issue, but this PR still adds other database engines to the protocol conversion func, so I'm keeping that set of changes.
Other than that, this PR fixes a related (unreported) issue I came across where a user can try to test connection an Aurora Postgres without a "Database name" - that doesn't work because Aurora postgres requires a database name just like other Postgres protocol engines:
So this PR makes database name a required field in the connection tester for any "postgres" protocol db.
public-teleport-github-review-bot[bot]
commented
2 minutes ago
Closes #47206
These changes were originally made to fix #47206. The fix for #47206 was preempted by https://github.com/gravitational/teleport/pull/48811 fixing a duplicate issue, but this PR still adds other database engines to the protocol conversion func, so I'm keeping that set of changes.
47206
Other than that, this PR fixes a related (unreported) issue I came across where a user can try to test connection an Aurora Postgres without a "Database name" - that doesn't work because Aurora postgres requires a database name just like other Postgres protocol engines:
So this PR makes database name a required field in the connection tester for any "postgres" protocol db.