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

[kbn/plugin-helpers/build] copy the public assets of a plugin #83458

Merged
merged 3 commits into from
Nov 16, 2020

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 Team label for 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
…c#83458)

Co-authored-by: spalger <spalger@users.noreply.github.com>
spalger added a commit to spalger/kibana that referenced this pull request Nov 16, 2020
…c#83458)

Co-authored-by: spalger <spalger@users.noreply.github.com>
spalger added a commit that referenced this pull request Nov 16, 2020
…83458) (#83494)

Co-authored-by: spalger <spalger@users.noreply.github.com>
spalger added a commit that referenced this pull request Nov 16, 2020
…83458) (#83493)

Co-authored-by: spalger <spalger@users.noreply.github.com>

Co-authored-by: spalger <spalger@users.noreply.github.com>
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 Team label for 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