mantl / mesos-consul

Mesos to Consul bridge for service discovery
Apache License 2.0
338 stars 95 forks source link

Add support for cert and cacert options #108

Closed mhamann closed 7 years ago

mhamann commented 7 years ago

Even though the options existed in the docs and config, support for the consul-ssl-cert and consul-ssl-cacert options were missing. This is a first pass at rectifying that.

Fixes #107.

mhamann commented 7 years ago

@ChrisAubuchon PTAL

ChrisAubuchon commented 7 years ago

@mhamann I no longer have commit rights to this repo.

mhamann commented 7 years ago

@BrianHicks are you able to look at merging this? Or is there someone who can? Is anyone maintaining this further?

a-nldisr commented 7 years ago

Any updates? Is this repo still maintained?

Theaxiom commented 7 years ago

@ChrisAubuchon Would you like commit rights?

Theaxiom commented 7 years ago

Come join us in Mattermost @ChrisAubuchon (and anyone else who wants to) https://mattermost.dynamictivity.com/signup_user_complete/?id=pjksr6h977n4zxxyto3ojp8twa

We're getting the Mantl engines started once again. 👍

ChrisAubuchon commented 7 years ago

@Theaxiom gitlab is not sending me a confirmation email...

Theaxiom commented 7 years ago

@ChrisAubuchon mail queue is backed up just a little, you'll receive an email shortly. Someone generated a ton of e-mail notifications.

Theaxiom commented 7 years ago

@ChrisAubuchon Can you please send me an e-mail directly and I will reply with your confirmation link. My e-mail address is travis.rowland@dynamictivity.com

Theaxiom commented 7 years ago

@ChrisAubuchon I just upgraded my transactional e-mail plan, it will increase my hourly send limit. Sorry for the inconvenience. 👍