Skip to content
This repository has been archived by the owner on Dec 2, 2024. It is now read-only.

Add support for unique master and follower certificates #183

Open
izgeri opened this issue Jul 8, 2020 · 1 comment
Open

Add support for unique master and follower certificates #183

izgeri opened this issue Jul 8, 2020 · 1 comment

Comments

@izgeri
Copy link
Contributor

izgeri commented Jul 8, 2020

Is your feature request related to a problem? Please describe.

Migrated from pivotal-cf/docs-cyberark-conjur-service-broker#21 as reported by @whip113

Currently, the tile config only has one field for importing the SSL certificate. The recommendations for certificates is to use one certificate for the master VIP and a different certificate for the follower VIP. The tile config allows us to configure both the master URL and the follower URL, but when using different certs for each the lack of a field to store the follower certificate means you can't actually use the follower URL.

The configuration for the tile relates to this service broker. At current, the service broker supports specifying the DAP master URL and a (read-only) follower URL - but it only allows specifying one certificate, when (as noted above):

The recommendations for certificates is to use one certificate for the master VIP and a different certificate for the follower VIP.

Describe the solution you would like

At current the service broker supports supplying the following relevant configuration environment variables:

  • CONJUR_SSL_CERTIFICATE
  • CONJUR_APPLIANCE_URL
  • CONJUR_FOLLOWER_URL

We could add another parameter

  • CONJUR_FOLLOWER_SSL_CERTIFICATE

And if set, we could use this alternate certificate when we instantiate the read-only API:

ConjurClient.new.api(ConjurClient.application_conjur_url)

Additional context

Once this change is made, we will also need to update the VMWare Tanzu tile definition to accept this parameter and properly configure the service broker with the CONJUR_FOLLOWER_SSL_CERTIFICATE env var.

@izgeri izgeri added this to the There is an updated release of the TAS tile - Milestone 2 milestone Aug 6, 2020
@izgeri izgeri removed this from the There is an updated release of the TAS tile - Milestone 2 milestone Nov 19, 2020
@izgeri izgeri added the on-hold label Feb 10, 2021
@tarnowsc
Copy link

fixed in the new version

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

Successfully merging a pull request may close this issue.

3 participants