-
Notifications
You must be signed in to change notification settings - Fork 843
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
Replace remark-parse
with remark-parse-no-trim
#6482
Conversation
- in order to remove `trim@0.0.1` security warning that downstream consumers receive
Preview documentation changes for this PR: https://eui.elastic.co/pr_6482/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great! I QA'd (if that's even a word) with the Markdown Editor and Markdown Format pages in the PR preview and in prod.
I'm gonna verb it as one even if it isn't! 😆 Thanks Bree! |
Preview documentation changes for this PR: https://eui.elastic.co/pr_6482/ |
## Summary `eui@71.0.0` ⏩ `eui@72.0.0` --- ## [`72.0.0`](https://github.com/elastic/eui/tree/v72.0.0) - Added the `customQuickSelectRender` render prop to `EuiSuperDatePicker`, which allows customizing the Quick Select popover ([#6382](elastic/eui#6382)) - `EuiFilePicker` styles have been updated to look more like an interactive element. ([#6479](elastic/eui#6479)) - Added a third argument to `EuiSelectable`'s `onChange` callback. The single `option` object that triggered the `onChange` event is now also passed to consumers with its most recent `checked` state ([#6487](elastic/eui#6487)) **Bug fixes** - `EuiTabs` now passes `size` and `expand` to all children using a React context provider. ([#6478](elastic/eui#6478)) - Fixed security warnings caused by `trim@0.0.1` sub-dependency ([#6482](elastic/eui#6482)) **Breaking changes** - Removed `size` and `expand` props from `EuiTab` ([#6478](elastic/eui#6478)) ## [`71.1.0`](https://github.com/elastic/eui/tree/v71.1.0) **Deprecations** - Renamed `EuiPageSideBarProps` to `EuiPageSideBarProps_Deprecated`, to reduce usage/confusion with `EuiPageSidebar` ([#6468](elastic/eui#6468)) Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Summary
This PR does not fully address #5543, but resolves the underlying issue being described (security issues caused by
remark-parse
's use oftrim@0.0.1
).Simply upgrading
remark-parse
is non-trivial due to all the reasons outlined in #5543 (comment), but it looks like GitHub's docs team at some point released a fork ofremark-parse@8.0.3
without the trim dependency/security warnings, which is exactly what we need for an interim solution.QA
General checklist