-
Notifications
You must be signed in to change notification settings - Fork 29
chore: 🤖 upgrade got package to solve vulnerability #2885
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
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
I did test build and run (smoke test) Admin and Desktop UI, and they work as expected.
Awesome work Lis!! Thank you for taking care of this vulnerability. 🙌 💪
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.
How should I be confirming ember-try
still works properly?
@@ -48,7 +48,8 @@ | |||
"node-gyp": "^10.0.0", | |||
"ansi-html": "^0.0.9", | |||
"fireworm>async": "^2.6.4", | |||
"ember-cli-babel>@babel/runtime": "7.27.6" | |||
"ember-cli-babel>@babel/runtime": "7.27.6", | |||
"package-json>got": "^14.4.7" |
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.
Should we have just overriden package-json to 10+ instead since it looks like they stop using got
?
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.
good point, upgrading package-json is cleaner solution.
also, i am attempting to run ember try
in the addons that use it and i am getting an error (same error on main
) im going to look into that.
Description
Dependency tree:
got@9.6.0
->package-json@6.5.0
->ember-try-config@4.0.0(encoding@0.1.13)
->ember-try@3.0.0(encoding@0.1.13)
Attempting to upgrade ember-try did not cause
got
to update to a non-vulnerable version.ember-try-config
is also already at newest released version andpackage-json
only goes up to 6.5 for v6. Therefore, as a last resort we forcegot
to resolve to newest, non-vulnerable version.Resolves alert #184
How to Test
Desktop release test run
Checklist