portworx / kvdb

Generic Key-Value interface
Apache License 2.0
40 stars 12 forks source link

Consul kvdb doesn't respect protocol scheme #11

Open mattlqx opened 6 years ago

mattlqx commented 6 years ago

When this is set in the config.json:

    "kvdb": [
        "consul:https://mycluster"
    ],

It still appears to use http to connect to the host. I'm guessing because it appears hardcoded here: https://github.com/portworx/kvdb/blob/master/consul/kv_consul.go#L123

It would be highly desirable to support encryption to the KV store.

sangleganesh commented 6 years ago

Thanks for reporting the issue, will try to fix it.