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

[Kerberos] Add stack documentation for Kerberos realm #98

Merged
merged 5 commits into from
Aug 20, 2018

Conversation

bizybot
Copy link
Contributor

@bizybot bizybot commented Aug 7, 2018

This commit adds documentation for Kerberos realm in
stack-docs. It briefly talks about Kerberos authentication,
its support and then points to settings and configuration
documentation.

This commit adds documentation for Kerberos realm in
stack-docs. It briefly talks about Kerberos authentication,
its support and then points to settings and configuration
documentation.
Copy link
Contributor

@lcawl lcawl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made one small edit, then copied some additional content from elastic/elasticsearch#32662


[[kerberos-terms]]
==== Key concepts

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added this content here instead of elastic/elasticsearch#32662, since it seems like a better fit with the overview info. If you disagree, let me know!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wasn't sure about this when I added to the configuring Kerberos section as most of the action was there. But looking at it now makes sense here as it is an introduction part and helps to understand key concepts before moving on to the configuration part. Thank you.


`instance` is an optional string that qualifies the primary and is separated
by a slash(`/`) from the primary. For a user, usually it is not used; for hosts,
it is the fully qualified domain name of the host.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is unclear why hosts are referenced here. Is that related to service principals?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not necessary but usually host is for service principals. Even users can have host FQDN as instance part of a principal name. The definition is as per MIT Kerberos documentation and I prefer to keep it the same. Thank you.

@bizybot bizybot merged commit b446630 into elastic:master Aug 20, 2018
bizybot added a commit that referenced this pull request Aug 20, 2018
This commit adds documentation for Kerberos realm in
stack-docs. It briefly talks about Kerberos authentication,
terminology, its support and then points to settings and 
configuration documentation.
bizybot added a commit that referenced this pull request Aug 20, 2018
This commit adds documentation for Kerberos realm in
stack-docs. It briefly talks about Kerberos authentication,
terminology, its support and then points to settings and 
configuration documentation.
@bizybot bizybot deleted the kerberos-stack-documentation branch August 21, 2018 06:36
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