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

Can ansi-html dependency be shifted to ansi-html-community to address advisory? #492

Open
lupestro opened this issue Oct 13, 2021 · 2 comments · May be fixed by #501
Open

Can ansi-html dependency be shifted to ansi-html-community to address advisory? #492

lupestro opened this issue Oct 13, 2021 · 2 comments · May be fixed by #501

Comments

@lupestro
Copy link

ansi-html has a high priority advisory against it for uncontrolled resource consumption and has had no new versions since 2017 and no response from the author. ansi-html-community is a fork of it that issued a 0.0.8 release in early September to address the issue. See PR on the original repo. Can the dependency be shifted to the fork?

@ohsnapitscolin
Copy link

It looks like the PR above has been merged and a new version of ansi-html has been released. Can the ansi-html dependency in this repo be updated to use the new version?

@gorner gorner linked a pull request Sep 30, 2022 that will close this issue
@gorner
Copy link

gorner commented Apr 5, 2024

Reposting a comment from the above-linked PR as a PSA for anyone this still affects:

Unfortunately this project seems to have remained dormant as the Ember community has shifted their focus to building out Embroider as the eventual replacement for Broccoli.

For now I would recommend specifying the updated version as an override in your package.json. Assuming you're using NPM, this would be:

"overrides": {
  "ansi-html": "^0.0.8"
}

(If you're using Yarn, use "resolutions" instead of "overrides".)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants