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

[6.x] Fix: plugin api route with security enabled (#23334) #23402

Merged
merged 1 commit into from
Sep 21, 2018

Conversation

cqliu1
Copy link
Contributor

@cqliu1 cqliu1 commented Sep 21, 2018

Backports the following commits to 6.x:

Closes elastic#23266

This is more of a quick fix than the final solution. The issue was that Canvas tries to check the plugins API without checking to see if the user it logged in. As a result, instead of the plugins response, it gets the HTML from the login page and that causes an error to be thrown when attempting to parse the results.

For now, this PR just disables the auth requirement on the Canvas plugin API endpoint.
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@cqliu1 cqliu1 merged commit 61b67e3 into elastic:6.x Sep 21, 2018
@cqliu1 cqliu1 deleted the backport/6.x/pr-23334 branch October 1, 2018 23:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants