Skip to content

Conversation

@spalger
Copy link
Contributor

@spalger spalger commented Nov 16, 2020

Fixes #83387

The updates to the plugin helpers in #75019 added a filter for the public directory when copying the server files for third-party plugins. This excluded the public/assets directory which the server exposes to the front-end at {basePath}/plugins/{id}/assets/{path}. This copies those files over as a new step in the build process.

@spalger spalger added Team:Operations Kibana-Operations Team v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.10.0 v7.11.0 labels Nov 16, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

Copy link
Member

@mistic mistic left a comment

Choose a reason for hiding this comment

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

LGTM

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@spalger spalger merged commit 859ba3e into elastic:master Nov 16, 2020
@spalger spalger deleted the fix/kp-plugin-public-asset-build branch November 16, 2020 22:08
spalger added a commit to spalger/kibana that referenced this pull request Nov 16, 2020
spalger added a commit to spalger/kibana that referenced this pull request Nov 16, 2020
spalger added a commit that referenced this pull request Nov 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes Team:Operations Kibana-Operations Team v7.10.0 v7.11.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[plugin] Public assets no longer exposed

4 participants