readrops / Readrops

Android multi-services RSS client
GNU General Public License v3.0
272 stars 20 forks source link

Readrops asks for too much articles, overloads the server and no article is retrieved #116

Open Massedil opened 3 years ago

Massedil commented 3 years ago

I wanted to test your app with my FreshRSS account but I can't use it because it seems that the app request too much articles in one request. My server crashes with a PHP Fatal error: Allowed memory size of 134217728 bytes exhausted.

Logs :

"GET /api/greader.php/reader/api/0/tag/list?output=json HTTP/1.1" 200 7852 "-" "okhttp/4.9.1"
"GET /api/greader.php/reader/api/0/subscription/list?output=json HTTP/1.1" 200 151146 "-" "okhttp/4.9.1"
"GET /api/greader.php/reader/api/0/stream/contents/user/-/state/com.google/reading-list?xt=user%2F-%2Fstate%2Fcom.google%2Fread&xt=user%2F-%2Fstate%2Fcom.google%2Fstarred&n=5000 HTTP/1.1" 500 240 "-" "okhttp/4.9.1"

I reported the problem here and it seems that it would be better to request less articles in the retrieve request.

@Frenzie pointed an helpful comment here.

Shinokuni commented 3 years ago

Indeed, Readrops fetches 5000 unread items at initial synchronization, which can be too much for some servers. I set this number for two reasons :

With fetching like only 1000 thousands items, there is a risk that some feeds won't have any unread items and Readrops currently can't fetch items by feed or do any kind of pagination.

Daniel15 commented 3 years ago

Try increase the memory limit in php.ini