-
-
Notifications
You must be signed in to change notification settings - Fork 22.2k
deps: qs@^6.14.1 #6972
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
deps: qs@^6.14.1 #6972
Conversation
|
Well done! |
|
@sergio-tao |
|
Hello! Is there a 4.x release planned that will include this fix soon? Thanks! |
|
Could we please have a patch version for express containing this? |
|
@serhalp @jaenster A fresh To update: (The body-parser update may not be needed depending on your version, but older versions of body-parser had qs pinned so it's good to include.) It occurs to me though that I may be too used to managing dependencies this way to understand why others want to solve it with a patch release. Why do you ask for a patch release? Is there something causing you pain here that a patch would solve? Tooling yelling at you? |
|
That works if you use qs and body-parser directly. But, we dont, we use nestjs express, which uses express, which uses these. But, i might be that im the inexperienced idiot here, so please teach me if im doing it wrong and can solve it without an override. Context, we use pnpm. |
Nest v10.4.22 includes updates to
pnpm has |
Port for #6969