Skip to content

Add token based authn documentation in stack docs #195

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

Merged

Conversation

bizybot
Copy link
Contributor

@bizybot bizybot commented Feb 1, 2019

This commit adds documentation for token-based
authentication services like token-service and
api-key-service. The overview section mentions
the token-based authentication services and a separate
page has information on token-service and api-key-service
mentioned in the order they are consulted for authentication.
The links for the API keys from Elasticsearch reference are not yet
on master but on the api-key-service feature branch.

This commit adds documentation for token based
authentication services like token-service and
api-key-service.
jkakavas and others added 4 commits February 1, 2019 22:24
- add example usage for Authorization headers of Bearer
and ApiKey schemes
@jaymode
Copy link
Member

jaymode commented Feb 4, 2019

I'm fine with the content, but I will let @lcawl have the final say

Copy link
Member

@jkakavas jkakavas left a comment

Choose a reason for hiding this comment

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

LGTM too granted that @lcawl is satisfied :)

@lcawl
Copy link
Contributor

lcawl commented Feb 5, 2019

I copied the intro from the new token-based authentication page to the existing "realms" page too.

@bizybot bizybot changed the base branch from api-key-service-stack-documentation to master February 6, 2019 05:27
@bizybot bizybot changed the base branch from master to api-key-service-stack-documentation February 6, 2019 05:30
@bizybot bizybot changed the base branch from api-key-service-stack-documentation to master February 6, 2019 05:47
@bizybot
Copy link
Contributor Author

bizybot commented Feb 6, 2019

Hi @lcawl, I have accepted your suggested changes and addressed the review comments.
Could you please review these changes? Thank you for your review and time.

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.

LGTM and builds successfully

@bizybot bizybot merged commit d37a49e into elastic:master Feb 6, 2019
bizybot added a commit to bizybot/stack-docs that referenced this pull request Feb 7, 2019
This commit adds documentation for token-based
authentication services like token-service and
api-key-service.
bizybot added a commit that referenced this pull request Feb 7, 2019
This commit adds documentation for token-based
authentication services like token-service and
api-key-service.
bizybot added a commit to bizybot/stack-docs that referenced this pull request Feb 8, 2019
This commit adds documentation for token-based
authentication services like token-service and
api-key-service.
bizybot added a commit that referenced this pull request Feb 8, 2019
This commit adds documentation for token-based
authentication services like token-service and
api-key-service.
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.

4 participants