Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve user experience for cloud strict encrypt connector when configuring unsecured connection #16664

Closed
rodireich opened this issue Sep 13, 2022 · 2 comments · Fixed by #17299

Comments

@rodireich
Copy link
Contributor

rodireich commented Sep 13, 2022

Tell us about the problem you're trying to solve

The strict encrypt postgres and mysql connectors correctly fail configuration check if the connection is effectively not guaranteed to be always encrypted (e.g with No Tunnel and with ssl set to "preferred").
Currently an error saying "Unsecured connection not allowed" is shown.

Improve user experience to reduce support calls from cloud customers.

Describe the solution you’d like

We can add a more informative message. Suggested: "If you do not have SSH tunneling set up, please use one of these SSL modes:….".
In addition, if spec UI allows we can apply form validation and mark setting as red, similarly to how a missing host or database name is handled:
Screen Shot 2022-09-13 at 1 15 55 PM
This can provide visual guidance to users.
Need to see if spec allows creating a logic based on both tunnel and ssl setting.

Describe the alternative you’ve considered or used

A clear and concise description of any alternative solutions or features you've considered or are using today.

Additional context

Add any other context or screenshots about the feature request here.

Are you willing to submit a PR?

Remove this with your answer :-)

@rodireich
Copy link
Contributor Author

Comment from @arimbr :

I think one way to make it less confusing for users would be to select the require option by default on Cloud?

@rodireich
Copy link
Contributor Author

Added to error message:
Screen Shot 2022-09-27 at 10 23 57 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants