-
Notifications
You must be signed in to change notification settings - Fork 59
Add SSL support #43
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
Comments
Hi, |
It's not supported yet. The driver could do it, but we need to add an API to use SSL in our client here as well. |
Ok, thanks for your response. Do you have any idea when this feature will be available ? |
I've just encountered same problem. It's impossible to run on Heroku w/o SSL :c this line shouldbe altered https://github.com/vert-x3/vertx-mysql-postgresql-client/blob/issues/sql-options/src/main/java/io/vertx/ext/asyncsql/impl/BaseSQLClient.java#L105 |
Now can be used with Heroku, as heroku supports only SSL
Now can be used with Heroku, as heroku supports only SSL
Is there any chance that #88 will be merged in any time soon? |
can we close this issue then ? |
Yes! |
I know this is closed and the PR has been merged. When will the new version be released with this feature? |
@jbkoh soon hopefully, it's being staged atm |
Thanks. I was wondering if I should start from java.sql directly or this. |
what are you aiming for ? |
Just want to make a Vertx service communicate with (or query to) a remote PostgreSQL with SSL enabled. Are there other solutions for this? |
@vietj Thanks for the pointer. I see that you are the main author of both this and that. Could you briefly introduce what the difference is? |
the client has been developed to be a client for postgres: it implements a bit more than the actual vertx-mysql-postgresql-client like batching, notifications, java 8 dates. |
Thanks for the prompt response. I will stick to the later. Thank you. |
It can be possible to use SSL encrypted connections with the newest driver version (0.2.19). I'll upgrade to that version shortly (that will fix another bug regarding
NUMERIC
columns) and then we are able to add SSL support.The text was updated successfully, but these errors were encountered: