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

[1.3] [CVE-2023-26136] Bump word-wrap from 1.2.3 to 1.2.4 #5002

Merged
merged 2 commits into from
Sep 14, 2023

Conversation

ananzh
Copy link
Member

@ananzh ananzh commented Sep 13, 2023

Issue Resolve:

CVE-2023-26136

Backport PR:

#4589

Check List

  • All tests pass
    • yarn test:jest
    • yarn test:jest_integration
    • yarn test:ftr
  • New functionality includes testing.
  • New functionality has been documented.
  • Update CHANGELOG.md
  • Commits are signed per the DCO using --signoff

Issue Resolve:
CVE-2023-26136

Backport PR:
opensearch-project#4589

Signed-off-by: ananzh <ananzh@amazon.com>
@codecov
Copy link

codecov bot commented Sep 13, 2023

Codecov Report

Merging #5002 (fbea0a9) into 1.3 (22657d7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              1.3    #5002   +/-   ##
=======================================
  Coverage   67.50%   67.50%           
=======================================
  Files        3044     3044           
  Lines       58692    58692           
  Branches     8902     8902           
=======================================
  Hits        39619    39619           
  Misses      16925    16925           
  Partials     2148     2148           
Flag Coverage Δ
Linux 67.45% <ø> (+<0.01%) ⬆️
Windows 67.45% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

@ananzh ananzh added v1.3.13 cve Security vulnerabilities detected by Dependabot or Mend labels Sep 14, 2023
Copy link
Member

@joshuarrrr joshuarrrr left a comment

Choose a reason for hiding this comment

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

Looks good, but changelog entry in in the wrong place.

CHANGELOG.md Outdated
@@ -87,6 +87,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
- [CVE-2022-25758][1.x] Bump scss-tokenizer from 0.3.0 to 0.4.3 ([#3789](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/3789))
- [CVE-2021-3803][1.x] Bump nth-check from 1.0.2 to 2.0.1 ([#3745](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/3745))
- Bump highlight.js from 9.18.5 to 10.7.3 to solve security concerns ([#4062](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/4062))
- [CVE-2023-26136] Bump `word-wrap` from `1.2.3` to `1.2.4` ([#5002](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/5002))
Copy link
Member

Choose a reason for hiding this comment

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

wrong section - should be under L10

Signed-off-by: ananzh <ananzh@amazon.com>
@ananzh ananzh merged commit 7b922e8 into opensearch-project:1.3 Sep 14, 2023
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cve Security vulnerabilities detected by Dependabot or Mend v1.3.13
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants