/utils

add SecureTLSConfig function #208

Merged
merged 1 commit into from Apr 28, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@natefinch @ericsnowcurrently @jujubot
@natefinch
Contributor

natefinch commented Apr 28, 2016

This is a helper function for use in juju that returns a tls.Config that is secure against known cryptographic vulnerabilities and ensures Juju is always using the most secure connections possible.

Verified with Security teams and the web teams that this configuration is workable with known clients and secure.
@ericsnowcurrently
Contributor

ericsnowcurrently commented Apr 28, 2016

You know why this didn't make its way over to reviewboard?
@ericsnowcurrently
Contributor

ericsnowcurrently commented Apr 28, 2016

LGTM

Thanks for the clear, detailed comments.
@natefinch
Contributor

natefinch commented Apr 28, 2016

dunno why
@natefinch
Contributor

natefinch commented Apr 28, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Apr 28, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju-utils

@jujubot jujubot merged commit b28c9ba into juju:master Apr 28, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment