Removing http & https web request library dependencies

rbren / rss-parser

A lightweight RSS parser, for Node and the browser

MIT License

1.38k stars 209 forks source link

Removing http & https web request library dependencies #222

Open gaodeng opened 2 years ago

gaodeng commented 2 years ago

Most projects using this library should already have their own network request library, so there doesn't seem to be much need to add the parseURL API. I suggest removing this method to remove the web request library dependency

rbren commented 2 years ago

This seems like a good idea to me. The HTTP logic has caused a lot of issues.

eviltik commented 2 years ago

https://github.com/rbren/rss-parser/pull/228