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.

Sign up for GitHub

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

Jump to bottom

Support WHATWG URL #69

Open
nezed opened this issue Sep 8, 2017 · 5 comments
Open

Support WHATWG URL #69

nezed opened this issue Sep 8, 2017 · 5 comments

Comments

@nezed
Copy link

nezed commented Sep 8, 2017 •
edited

It seems like there is no activity in

And i propose to replace node-url with
@mightyiam
Copy link

url-parse is so much more popular. Has a fancy test suite.

@sawyerh
Copy link

sawyerh commented Jan 23, 2018

Node v7+ includes a browser-compatible URL class. For folks who want to use the built-in class, is there a way to ignore the url dependency installed by node-libs-browser?

@DomEscobar DomEscobar mentioned this issue Jan 31, 2018
Closed
@sindresorhus sindresorhus mentioned this issue May 30, 2018
Closed
@jaibatrik
Copy link

@sawyerh I got around this by using the universal-url package.

olizilla added a commit to multiformats/js-uri-to-multiaddr that referenced this issue Oct 23, 2018
@olizilla
fix: origin relative api detection
82540e7 
- Use the gloabl URL object. We hit webpack/node-libs-browser#69
- This will only work in the browser and node >= 10 now.

License: MIT
Signed-off-by: Oli Evans <oli@tableflip.io>
@alexander-fenster alexander-fenster mentioned this issue Jan 7, 2019
Merged
@dmytrobanasko dmytrobanasko mentioned this issue Feb 11, 2019
Closed
@wh1tecat-nya
Copy link

in node v11, url.parse is deprecated.
so recommended url.URL class.
we need change package or update node-url package.

@niksy
Copy link

niksy commented Nov 13, 2021

If anyone is looking for up-to-date implementation, I’ve created package which includes latest Node features.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@sawyerh @niksy @mightyiam @nezed @jaibatrik @wh1tecat-nya