Skip to content
This repository has been archived by the owner on Aug 15, 2024. It is now read-only.

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
Loading

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 <[email protected]>
@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 subscribe to this conversation on GitHub. Already have an account? Sign in.
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