Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

go-nsq: add tls-min-version config option #98

Merged
merged 1 commit into from
Dec 10, 2014
Merged

go-nsq: add tls-min-version config option #98

merged 1 commit into from
Dec 10, 2014

Conversation

twmb
Copy link
Contributor

@twmb twmb commented Dec 10, 2014

This adds a new tls-min-version flag that enables nsqd users
to specify a min tls version acceptable. The arguments to this
flag, , follow the format of Version in the tls package.

This commit parallels nsqio/nsq#513 for issue nsqio/nsq#512.

This adds a new tls-min-version flag that enables nsqd users
to specify a min tls version acceptable. The arguments to this
flag, <v>, follow the format of Version<v> in the tls package.

This commit parallels nsqio/nsq#513 for issue nsqio/nsq#512.
@twmb
Copy link
Contributor Author

twmb commented Dec 10, 2014

RFR @mreiferson

@mreiferson
Copy link
Member

LGTM

mreiferson added a commit that referenced this pull request Dec 10, 2014
go-nsq: add tls-min-version config option
@mreiferson mreiferson merged commit ba766e5 into nsqio:master Dec 10, 2014
@twmb twmb deleted the tls_min_version_512 branch December 10, 2014 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants