search

jfroelich / rss-reader

A simple Chrome extension for viewing RSS feeds
Other
10 stars 0 forks source link

[net] convert net module into a library #743

Closed jfroelich closed 5 years ago

jfroelich commented 5 years ago

net is mostly generic, not app-specific. the only app-specific behavior tends to be the fetch policy. instead the generic lib could just provide policy param that defaults to permit all. then the app wrapper lib can substitute in its own app-specific policy.

part of goal of moving generic libs out of core, so that core is only the app-specific functionality.

it is annoying to admit that I had previously accomplished this, and lost this design when deciding to try and remove the delineation between what functionality is part of the app and what is part of some lib.

jfroelich commented 5 years ago

Done, except for fetch-feed due to its highly specialized behavior and reliance on core modules