prof18 / RSS-Parser

A Kotlin Multiplatform library to parse a RSS Feed
Apache License 2.0
509 stars 129 forks source link

Use Call.Factory instead of OkHttpClient #98

Closed yschimke closed 1 year ago

yschimke commented 2 years ago

Is your feature request related to a problem? Please describe.

Passing in OkHttpClient is awkward if the app uses Call.Factory instead.

Describe the solution you'd like

Call.Factory is the typical api for libraries like Coil and Retrofit

https://github.com/square/retrofit/blob/9b32c965ef31275627e35a862f584bcc7b9fa4e5/retrofit/src/main/java/retrofit2/Retrofit.java#L69 https://github.com/coil-kt/coil/blob/481fa962f29c137eab07aca91944f783208acab7/coil-base/src/main/java/coil/ImageLoader.kt#L118

Describe alternatives you've considered

Using a raw OkHttpClient.