-
Notifications
You must be signed in to change notification settings - Fork 313
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Added support for tls min version and tls cipher suites
- Loading branch information
1 parent
ddbcf2d
commit c7f4571
Showing
2 changed files
with
9 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
c7f4571
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jasonneurohr can you update the documentation? Our consul playbook doesn't have
vault_tls_prefer_server_cipher_suites
defined and this commit broke the build.Did you perhaps mean to use
consul_tls_prefer_server_cipher_suites
? (note theconsul
prefix)c7f4571
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @issmirnov - good pickup. I'm submitting a fixed version now which also includes the documentation updates.
c7f4571
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request submitted - #302