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

Replace remark-parse with remark-parse-no-trim #6482

Merged
merged 3 commits into from
Dec 14, 2022

Conversation

cee-chen
Copy link
Contributor

@cee-chen cee-chen commented Dec 14, 2022

Summary

This PR does not fully address #5543, but resolves the underlying issue being described (security issues caused by remark-parse's use of trim@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 of remark-parse@8.0.3 without the trim dependency/security warnings, which is exactly what we need for an interim solution.

QA

General checklist

  • A changelog entry exists and is marked appropriately

- in order to remove `trim@0.0.1` security warning that downstream consumers receive
@cee-chen cee-chen added dependencies Pull requests that update a dependency file tech debt labels Dec 14, 2022
@cee-chen cee-chen requested a review from breehall December 14, 2022 20:16
@kibanamachine
Copy link

Preview documentation changes for this PR: https://eui.elastic.co/pr_6482/

Copy link
Contributor

@breehall breehall left a 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.

@cee-chen
Copy link
Contributor Author

I QA'd (if that's even a word)

I'm gonna verb it as one even if it isn't! 😆 Thanks Bree!

@cee-chen cee-chen enabled auto-merge (squash) December 14, 2022 21:53
@kibanamachine
Copy link

Preview documentation changes for this PR: https://eui.elastic.co/pr_6482/

@cee-chen cee-chen merged commit 4edc93a into elastic:main Dec 14, 2022
@cee-chen cee-chen deleted the remark-trim-fix branch December 14, 2022 22:28
jbudz pushed a commit to elastic/kibana that referenced this pull request Dec 22, 2022
## 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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file tech debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants