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

Add auth support in Kibana #4597

Merged
merged 1 commit into from
Jul 4, 2017
Merged

Conversation

monicasarbu
Copy link
Contributor

@monicasarbu monicasarbu commented Jul 3, 2017

This PR adds support for SSL and basic authentication with username and password to connect to Kibana.

@monicasarbu monicasarbu added the in progress Pull request is currently in progress. label Jul 3, 2017
@monicasarbu monicasarbu force-pushed the secure_kibana_connection branch 2 times, most recently from 053aa24 to 7b51443 Compare July 3, 2017 21:34
@monicasarbu monicasarbu changed the title [WIP] SSL support in Kibana Add auth support in Kibana Jul 3, 2017
@monicasarbu monicasarbu force-pushed the secure_kibana_connection branch from 7b51443 to 80fbac0 Compare July 3, 2017 21:37
@monicasarbu monicasarbu force-pushed the secure_kibana_connection branch from eef0dd0 to 0367910 Compare July 3, 2017 21:47
@monicasarbu monicasarbu added review and removed in progress Pull request is currently in progress. labels Jul 3, 2017
@@ -17,6 +17,7 @@ https://github.com/elastic/beats/compare/v6.0.0-alpha2...master[Check the HEAD d
- Rename `kubernetes` processor to `add_kubernetes_metadata`. {pull}4473[4473]
- Rename `*.full.yml` config files to `*.reference.yml`. {pull}4563[4563]
- The `scripts/import_dashboards` is removed from packages. Use the `setup` command instead. {pull}4586[4586]
- Change format of the saved kibana dashboards to have a single JSON file for each dashboard {pull}4413[4413]
Copy link
Contributor

Choose a reason for hiding this comment

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

This changelog doesn't reflect the real change (TLS support), is that intended?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's by intention, as I forgot to update the changelog in the previous PR (#4413)

@monicasarbu monicasarbu force-pushed the secure_kibana_connection branch from 0367910 to 9c5122f Compare July 4, 2017 12:53
Copy link
Contributor

@exekias exekias left a comment

Choose a reason for hiding this comment

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

WFG

@tsg tsg merged commit a0f55eb into elastic:master Jul 4, 2017
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.

3 participants