Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

UseObject.assign instead of deprecatedutil._extend#1666

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
nikeee wants to merge2 commits intohttp-party:master
base:master
Choose a base branch
Loading
fromnikeee:patch-1

Conversation

@nikeee
Copy link

When building an app using Vite, there is this deprecationw arning when using Node.js 22:

[DEP0060] DeprecationWarning: The `util._extend` API is deprecated. Please use Object.assign() instead.

This PR resolves that.

Object.assign issupported since Node.js 4.

wojtekmaj, matt8707, danjoa, bayasdev, ipcjs, btea, ozooner, rickdoesburg, nastassia-martin, d-oliveros, and 59 more reacted with thumbs up emoji
Resolves:```[DEP0060] DeprecationWarning: The `util._extend` API is deprecated. Please use Object.assign() instead.```When using Node.js 22
@bayasdev
Copy link

+1

d-oliveros, TheJaredWilcurt, Auios, fabiandecuveland, josepcrespo, yakovenkodenis, and shriekdj reacted with thumbs up emojiwojtekmaj, Netail, kachkaev, and eps1lon reacted with thumbs down emoji

@Malwinderkaur
Copy link

When should we expect updated version with required changes for fix of this warning.

TheJaredWilcurt, kolesar-andras, lufinkey, IMSterling, and kiwijuicer reacted with thumbs up emoji

@wj17870569856
Copy link

When should we expect updated version with required changes for fix of this warning.

so, when we can use the updated version 😂

austinbiggs reacted with eyes emoji

@Auios
Copy link

Ping@indexzero
Are you able to merge this?

@cami-dev
Copy link

Any release happening for this?

rtconner, austinbiggs, isnolan, josepcrespo, and mschori reacted with eyes emoji

@austinbiggs
Copy link

What would it take to get this merged?

kachkaev, edobrb, niksy, cdeutsch, Netail, mikomagni, wojtekmaj, alcatraz627, sushitommy, srackley, and 3 more reacted with thumbs up emoji

@cami-dev
Copy link

cami-dev commentedJan 16, 2025
edited
Loading

Nice that this got merged. Would be awesome with a release now been a few years :)@jcrugzz is one in the works? or prob not happening? should we be using master instead?

kachkaev, Netail, isnolan, and ArGeoph reacted with confused emojiwojtekmaj, StorytellerCZ, av-k, isnolan, and aslansky reacted with eyes emoji

@lwr
Copy link

lwr commentedMay 7, 2025

Now still no any progress after a year. So we have to put this patch in preinstall script

['http-proxy/lib/http-proxy/index.js','http-proxy/lib/http-proxy/common.js',// any others].map(id=>require.resolve(id)).filter(file=>/_extend/.test(fs.readFileSync(file,'utf8'))).forEach(file=>{console.log('patching',file);fs.writeFileSync(file,fs.readFileSync(file,'utf8').replaceAll("require('util')._extend",'Object.assign'));})

@glenntws
Copy link

Half of all framework dev servers depend on this dependency and the fix is a few lines of codes with easy replaces and we still don't get an official patch release. wtf.

wojtekmaj, codedusting, and stalingino reacted with thumbs up emoji

@wojtekmaj
Copy link

TBH this issue has been patched on Vite's side over a year ago, and this week, theydropped dependency on http-proxy, switching to a maintained fork, http-proxy-3. Which I would recommend doing as well.

ipcjs, djgrant, and joshbuchea reacted with hooray emoji

@shriekdj
Copy link

when it will be merged

albertosantini and fahlisaputra reacted with thumbs up emojialbertosantini reacted with heart emoji

@tianyingchun
Copy link

any updates on this? long time no update & release

@Auios
Copy link

Ping@indexzero Are you able to merge this?

Gah, I dont even remember what I needed this PR merged for now lol

@KJ7LNW
Copy link

+1

Is this repository is no longer being maintained?

Are there alternative projects that should be used instead?

@xiaoxiyao
Copy link

xiaoxiyao commentedOct 22, 2025
edited
Loading

Are there alternative projects that should be used instead?

https://github.com/sagemathinc/http-proxy-3

Auios and joshbuchea reacted with heart emoji

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

3 more reviewers

@TheJaredWilcurtTheJaredWilcurtTheJaredWilcurt approved these changes

@NetailNetailNetail approved these changes

@shriekdjshriekdjshriekdj approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

16 participants

@nikeee@bayasdev@Malwinderkaur@wj17870569856@Auios@cami-dev@austinbiggs@lwr@glenntws@wojtekmaj@shriekdj@tianyingchun@KJ7LNW@xiaoxiyao@TheJaredWilcurt@Netail

[8]ページ先頭

©2009-2025 Movatter.jp