akrennmair / newsbeuter

Newsbeuter is an open-source RSS/Atom feed reader for text terminals.
http://www.newsbeuter.org/
MIT License
780 stars 100 forks source link

JSON-Api for tt-rss #43

Closed t-8ch closed 11 years ago

t-8ch commented 12 years ago

Since tt-rss v1.6.0 (released on 19.10.2012) it doesn't support the old GET based API anymore. The recommended way is now JSON.

Using current master (b7f4631fbf) with the new tt-rss make newsbeuter hangs on:

Loading URLs from Tiny Tiny RSS

Messages in log:

[..] loads of OpenSSL stuff
[2012-10-28 18:57:33] DEBUG: utils::retrieve_url(https://baseurl.of/tt-rss/api/?op=login&sid=&password=password&user=user): {"seq":null,"status":1,"content":{"error":"NOT_LOGGED_IN"}}
[2012-10-28 18:57:33] DEBUG: ttrss_api::run_op(login,...): reply = {"seq":null,"status":1,"content":{"error":"NOT_LOGGED_IN"}}

If you need a tt-rss instance to do debugging I would be glad to provide one.

skn commented 11 years ago

This is indeed a pity and I do think tt-rss was wrong to completely cut this API off (edit: I stand corrected, this support has been marked as obsolete for some time now) but now that it has, I hope newsbeuter can start using JSON too. At my side, I can't update tt-rss until then :(

skinner33 commented 11 years ago

I just wrote a patch (skinner33/newsbeuter@b6957af774a0cb49fcb08c7e25cdafb1e8feed3e) which moves newsbeuter to the new api.

The basic functionality seams to be given, although feeds from the category "uncategorized" appear twice. I will check tomorrow if this is a bug in the patch or the API.

skn commented 11 years ago

The patch works great. Tested on my updated 1.6.1 install of tt-rss. Thanks!

t-8ch commented 11 years ago

Works for me too, thanks @skinner33 and @akrennmair! Closing this.