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 hostname verification in TLS transport docs #2403

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

Document hostname verification in TLS transport docs #2403

ivankelly opened this issue Aug 20, 2018 · 1 comment
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

#1208 Added the feature.

Verify that it works, and add documentation to http://pulsar.apache.org/docs/en/security-tls-transport/

It is already mentioned in passing, but we should be more explicit.
@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 added a commit to ivankelly/pulsar that referenced this issue Oct 15, 2018
@ivankelly
Document TLS hostname verification option (apache#2403)
06208a3 
Inform the user why it is disabled by default and in which scenarios
they should enable it.

Master issue: apache#2403
@ivankelly ivankelly mentioned this issue Oct 15, 2018
Merged
@ivankelly
Copy link
Contributor Author

#2793 blocks this from being 100% completed.

@sijie sijie closed this as completed in c848860 Oct 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
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