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.

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

[Feature Request] Separate downloading and parsing #58

Open
lonix1 opened this issue May 12, 2024 · 1 comment
Open

[Feature Request] Separate downloading and parsing #58

lonix1 opened this issue May 12, 2024 · 1 comment

Comments

@lonix1
Copy link

lonix1 commented May 12, 2024

There seem to be a number of open issues (e.g. #55 #42 #29 #22) related to the downloading of a feed. One of them (#55) is quite important.

I think the library is excellent, but it does too much.

Parsing:
It would be nice if we could use it for parsing only, by supplying our own XML strings.

Downloading:
For those who want to use it for downloading as well, the existing methods could be used (so I'm not referring to a breaking change). However, the HttpClient should be initialised in a static constructor, so that only those users who want it will actually cause it to be created. Those who don't use it - and only use the parser - would not cause the client to be created.

@Roald87
Copy link

Roald87 commented Sep 11, 2024

I forked this repo and removed all the HttpClient stuff from it. The library only does parsing now. Also I ported to .NET 8.

https://github.com/Roald87/FeedReader

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants