Pretty sure it can be removed and cleaned up now.

Thanks Roger

On 25/05/17 02:22, roger peppe wrote:
I recently came across the code introduced by
https://github.com/juju/juju/pull/2294 which provides support for
reading extra certificates from /etc/juju/certs.d when connecting to
an API server. The PR description says:

"This feature is likely to be used when there is an agreed central JES
system that users can log in to without having to supply their own CA
certificate"

However we have gone in a different direction for that - controllers
can now use public certificates so no extra certificates are needed.

I can't see any references in the code to CreateCertPool or
paths.CertDir, so it seems like it's redundant. Can anyone think of a
reason this code can't be removed now?

   cheers,
     rog.


--
Juju-dev mailing list
Juju-dev@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju-dev

Reply via email to