horizon icon indicating copy to clipboard operation
horizon copied to clipboard

SSL option for RethinkDB connection

Open danielmewes opened this issue 9 years ago • 7 comments

https://github.com/rethinkdb/horizon/pull/507 added support for RethinkDB's user and password options, but didn't add a way for enabling SSL for the RethinkDB connection.

We'll need an option to pass the location of the RethinkDB SSL certificate to Horizon, and Horizon has to pass the certificate's file path to r.connect through the ssl option if specified.

This is required for connecting directly to Compose RethinkDB clusters for example.

danielmewes avatar Jul 27 '16 19:07 danielmewes

+1

geddski avatar Aug 02 '16 20:08 geddski

+1 For me as well. Unfortunately, this became a non-starter for me because my company has a secure protocol only policy. Hopefully, this can be addressed soon, because I can see allot of promise in Horizon.

Linicks avatar Aug 05 '16 20:08 Linicks

+1 I thought 2.0 was going to support SSL?

GeoffreyPlitt avatar Aug 10 '16 21:08 GeoffreyPlitt

We should put this into Horizon 2.1.0 (or maybe even still add it to 2.0.0 before we release the final version?)

danielmewes avatar Aug 10 '16 21:08 danielmewes

Cool!

GeoffreyPlitt avatar Aug 10 '16 22:08 GeoffreyPlitt

This was a problem for me as well, it's a quick fix to add a rdb_cert option in the horizon() constructor and then add ssl: { ca: this._cert } to the r.connect() call in https://github.com/rethinkdb/horizon/blob/next/server/src/reql_connection.js

b-d avatar Aug 17 '16 19:08 b-d

Is there an ETA yet on Horizon+Compose compatibility?

GeoffreyPlitt avatar Sep 21 '16 06:09 GeoffreyPlitt