-
Notifications
You must be signed in to change notification settings - Fork 106
Can't add new feed #732
Comments
Can you post the feed URL? |
The issue occurs with almost all feed, here an example: |
Works perfectly fine here. It could be that the number that you've entered in your config is bigger than a 32 or 64 bit integer, overflows and becomes a smaller or even negative number. Can you try to delete your config file to go back to the default? |
If that does not help please post more information about your server by following the guideline outlined in the readme |
After setting the config file to the default values the issue still occurs. I use on my server:
I use on my client:
This is my owncloud.log: ---- UPDATE ---- I found in the php log the following error: This may be related? |
Hm, actually no idea, also cc @fguillot Can you log out the max size, by putting this line error_log($config->getMaxSize()); // goes to your httpd error log and not the data/owncloud.log here: https://github.com/owncloud/news/blob/master/appinfo/application.php#L190 ? |
This is the result: that matches with the config file: |
The exception 'MaxSizeException' is caught in the file fetcher/feedfetcher.php I log the stack trace and this is the result: As you can see there is a curl error 'CURLE_WRITE_ERROR' (23) |
Do you have enough ram/disk space? |
Do you use the encryption app? |
I've ~90 Mb of free ram, and I use the encryption app on the current owncloud account. The hdd have a lot of Gb free. |
Can you disable the encryption app? News is not tested against it and I highly discourage from using it. |
I disabled the encryption and file lock apps, but the bug still persists. Seems to be a problem with the curl module, but i don't know what the 'CURLE_WRITE_ERROR' error means. |
In apps/news/vendor/fguillot/picofeed/lib/PicoFeed/Client/Curl.phpthe curl are set to use a callback function: As written in the documentation: "Your callback should return the number of bytes actually taken care of. If that amount differs from the amount passed to your callback function, it'll signal an error condition to the library. This will cause the transfer to get aborted and the libcurl function used will return CURLE_WRITE_ERROR." The question is: why the callback return the wrong length? |
You could try to log out if this section is ever run https://github.com/fguillot/picoFeed/blob/master/lib/PicoFeed/Client/Curl.php#L63 |
If it is not then we know it does not have anything to do with the maximum size |
I suspect that it could be a too little RAM issue. Can you try it on a system with 512mb free ram? |
I correctly log $length before the last return (https://github.com/fguillot/picoFeed/blob/master/lib/PicoFeed/Client/Curl.php#L67), so isn't a maximum size issue. I can't increase ram, my owncloud instance run in a vps. I can try to free other ram. |
RESOLVED In my PHP config was enabled the 'mbstring.func_overload' option, this broke some PHP code. Disabling that option has resolve the issue. Thanks for the support |
@Andreabont cool! Closing this. P.S.: btw, thanks for the detailed reporting. |
Hi,
I'm using Owncloud 8 with the last git version of the news app (commit 1029392)
When i try to add a new feed, the app fails with the error: "Bigger than maximum allowed size".
This is my config.ini:
autoPurgeMinimumInterval = 60
autoPurgeCount = 100
maxRedirects = 100
maxSize = 10000000000
exploreUrl =
feedFetcherTimeout = 60
Can you help me? Thank you.
The text was updated successfully, but these errors were encountered: