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.

Sign up for GitHub

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

Jump to bottom

Document TLS version and ciphersuite configuration for server and proxy #2402

Closed
ivankelly opened this issue Aug 20, 2018 · 1 comment
Closed

Document TLS version and ciphersuite configuration for server and proxy #2402

ivankelly opened this issue Aug 20, 2018 · 1 comment
Assignees
@ivankelly
Labels
doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. type/bug The PR fixed a bug or issue reported a bug

Comments

@ivankelly
Copy link
Contributor

#1225 added support for specifying the TLS version and ciphersuite to use when configuring the proxy and broker. However, this is not documented in the respective configuration files, nor in the user documents. We should add such docs (and verify that they take effect).
@ivankelly ivankelly mentioned this issue Aug 20, 2018
Closed
@sijie sijie added type/bug The PR fixed a bug or issue reported a bug doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. triage/week-34 labels Aug 22, 2018
ivankelly pushed a commit to ivankelly/pulsar that referenced this issue Nov 26, 2018
Documentation for TLS protocol version and ciphers
7e21733 
PR apache#1225 added the ability to configure the TLS protocol version and
cipher on the server side, but this was never added to the
documentation. This patch fixes that omission.

Issue: apache#2402
@ivankelly ivankelly mentioned this issue Nov 26, 2018
Merged
merlimat pushed a commit that referenced this issue Nov 26, 2018
@ivankelly @merlimat
Documentation for TLS protocol version and ciphers (#3057)
ff76bd7 
PR #1225 added the ability to configure the TLS protocol version and
cipher on the server side, but this was never added to the
documentation. This patch fixes that omission.

Issue: #2402
@sijie
Copy link
Member

sijie commented Nov 28, 2018

This issue was fixed by #3057

@sijie sijie closed this as completed Nov 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ivankelly ivankelly

Labels
doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. type/bug The PR fixed a bug or issue reported a bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ivankelly @sijie