-
Notifications
You must be signed in to change notification settings - Fork 259
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
App enters in UnsuccesfulResponseException
notification loop
#905
Comments
Hi, thanks for drawing attention to the problem. To be honest I've always been reluctant to developing Android UI, I guess it's the main reason this is still not addressed. Here are some thoughts for me or somebody else to implement:
For the 3rd point (the one that I really don't want to work on) I'm tempted to simply add two buttons: "copy failed items to clipboard" and "remove failed items". I wonder whether the current state of things warrants a solution even that poor. |
If that can help, I faced the same error a few times in previous days, everytime I had to uninstall the app and reinstall it because I had no clue what was suddenly wrong. Server logs (running Wallabag from Cloudron) :
Mobile app (Android) logs :
|
Have you solved this by increasing the allowed memory size for php on server side? |
Hi @Strubbl I'm not sure how much memory is needed for Wallabag ? with about 1.5 links it seems 650 MB is not enough ? I've seen people struggling with Wallabag even with 2GB memory allocated. |
Your server's memory limit for php seems to be at 256 MB. I have increased mine to 1024 MB. It is sufficient, but i almost never add PDFs to my wallabag. |
@Strubbl this seems unexpectedly low compared to what instructions are given to Cloudron for this app. Maybe the setting is for the whole image and not for PHP. The whole Cloudron template for this app is https://git.cloudron.io/cloudron/wallabag-app and I see in https://git.cloudron.io/cloudron/wallabag-app/-/blob/master/Dockerfile that the memory is hardcoded for PHP which could explain several of the memory issues I've encountered with this app from the beginning while I was always increasing the overall memory allocated to wallabag (the whole container). Regarding usage of Wallabag for bookmarking PDFs URLs, well, sometimes a document is only available as a PDF and you want to read it later and wallabag should be able to save just an URL to a PDF so I can read it later, at least that was one use case for me that I expect keeps being supported. EDIT : case opened at https://forum.cloudron.io/topic/10824/app-ran-out-of-memory-related-to-php-settings |
Yet another case of URL that causes issues on server is http://sebsauvage.free.fr/favs.html , it seems to take forever to process. No idea why |
Well http://sebsauvage.free.fr/favs.html is huge, nearly 30k lines of html. I managed to add it to my wallabag instance, using firefox's extension, not the android app. The http call timeout out but the backend process finished and the entry is available. About your concern about wallabag not being suited to your needs, Wallabag is more than just a way to saving links, it keeps a local copy of the page's content allowing you to read it later, possibly offline, it's also a way to make a snapshot of a page in case it disappear. Anyway, I've been using Wallabag both as a read-it-later and a "simple" link save for 8 years now and I almost never encountered cases where I could not add a link, I can't remember a specific one. On a final note, this issue is about the android app being resilient from server errors, not the server errors themselves which should be issued in the main repository IMO. |
I do care about the read-it later and archiving parts as well as for the automated tagging and other nice features. But there are some features I lack and had to take care of via integrations like archiving to wayback machine. Thank you for your feedback @Zayon ! |
Issue details
Duplicate?
Have you searched the issues of this repository if your issue is already known? yes
Actual behaviour
I tried to add an entry through the app, but my wallabag server crashed (error 500) because it could not insert the entry in the database due to the entry containing emojis and the database charset not being properly configured.
The app enters in a notification loop:
Every few seconds the app retry to add the entry and re triggers the notification. This is really annoying.
The only way to make it stop is to force stop the app from the android settings.
The app becomes unusable as long as the server is not fixed.
Expected behaviour
There should be a way to cancel an entry.
Steps to reproduce the issue
database_charset: utf8
instead ofdatabase_charset: utf8mb4
)Environment details
Logs
wallabag server
Please paste relevant wallabag server logs here (from file
<wallabag>/var/logs/prod.log
):Just any CRITICAL error log.
Web server
Please paste relevant web server logs (e.g. from nginx, Apache, …) here:
Your experience with wallabag Android app
Have you had any luck using wallabag Android app before? (Sometimes we get tired of reading bug reports all day and a lil' positive end note does wonders)
Can't live without it, wallabag would be unusable on mobile without it.
The text was updated successfully, but these errors were encountered: