Add native Miniflux account type with API key authentication#5335
Add native Miniflux account type with API key authentication#5335IngmarStein wants to merge 2 commits into
Conversation
Uses Miniflux's native REST API (/v1/*) instead of the Google Reader compatibility layer. Authenticates via X-Auth-Token header. - New AccountType.miniflux (24) with dedicated MinifluxAccountDelegate - MinifluxCaller: full API client for feeds, categories, entries, OPML - Version detection (GET /v1/version) gates /v1/entries/ids for v2.3.2+ with transparent fallback to /v1/entries pagination on older instances - macOS: credential sheet with API URL + API key fields - iOS: credential view controller with same - Asset catalogs: accountMiniflux icon in both Mac and iOS Closes Ranchero-Software#3593 Closes Ranchero-Software#4743
|
I'd love to see this merged, as I've just moved to Miniflux as a back-end for my feeds. |
|
Hey @brentsimmons, is this something you'd consider merging? I know that Miniflux can already be connected as a FreshRSS account, but this PR improves upon that in several ways:
|
|
Yeah, it would be nice to have this instead of having the other API poke at the database. My picoflux fork (which uses SQLite) works with the Reader API, but the native one is quite a bit more efficient, and it shows at the database level. |
|
Hi @IngmarStein — great to see this PR. I owe you an apology for not finding it sooner: I've been building a native Miniflux account type for NetNewsWire in parallel, over on Rather than run two competing PRs, I'd like to team up on a single effort. I feel like one well-tested PR seems like our best shot at getting Miniflux support landed. A few things my branch brings that might be useful to combine:
And I've already adopted two ideas from your PR into my branch — thank you for these:
One thing I deliberately didn't carry over is If you're open to it: take a look at my branch, poke holes in it, and let's jointly propose a merged version as the successor to this PR. Or just tell me to go fly a kite 😉 |
|
@dave-atx looks like you already incorporated the good parts of this PR into your branch. Happy to retire this one in favor of yours - maybe you get more traction! |
|
Sounds great... I opened a thread in the NetNewsWire discourse to discuss there. |
|
Alright. That link doesn't work for me, but there's also https://discourse.netnewswire.com/t/miniflux-support/37 |
Uses Miniflux's native REST API (/v1/*) instead of the Google Reader compatibility layer. Authenticates via X-Auth-Token header.
Closes #3593
Closes #4743