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

Allow ['*'] for actions whitelistedHosts #46384

Merged

Conversation

mikecote
Copy link
Contributor

In this PR, I'm changing the Joi validation for actions plugin config to allow xpack.actions.whitelistedHosts: ['*']. The underlying code & tests support it but the validation wasn't aligned.

@mikecote mikecote requested review from bmcconaghy and a team September 23, 2019 19:05
@mikecote mikecote self-assigned this Sep 23, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-stack-services

@mikecote mikecote changed the title Allow ['*'] for whitelistedHosts Allow ['*'] for actions whitelistedHosts Sep 23, 2019
@FrankHassanabad
Copy link
Contributor

Tested this out on my machine and this works as advertised to where I can start up and run with this in my kibana.dev.yml:

xpack.actions.whitelistedHosts: ['*']

👍 LGTM

Copy link
Contributor

@FrankHassanabad FrankHassanabad left a comment

Choose a reason for hiding this comment

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

I tested and this is LGTM to me.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Contributor

@bmcconaghy bmcconaghy left a comment

Choose a reason for hiding this comment

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

Code LGTM

@mikecote mikecote merged commit 03f8e9d into elastic:master Sep 24, 2019
mikecote added a commit to mikecote/kibana that referenced this pull request Sep 24, 2019
mikecote added a commit that referenced this pull request Sep 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Alerting release_note:skip Skip the PR/issue when compiling release notes review v7.5.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants