-
-
Notifications
You must be signed in to change notification settings - Fork 78
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
Synchronization fails when for instance DESCRIPTION is too large #1126
Comments
Thanks for the report!
This will only apply to the So we have two problems:
The latter shouldn't happen, at least there was no intention to change it and if it's true, the error should be shown again. |
Hi! Having done some tests on iCalendar, I found that As it was indicated, there are two problems and, as far as I understand, the second one is going to be fixed. Let us proceed to the first indicated problem. You said that you didn't know how to overcome the 1 MB IPC limit when talking to the calendar provider. It appears to me that, if we cut the
As a result, both the events although being modified reached the device. Could you share your opinion on the approach to overcome the 1 MB IPC limit and the patch implementing it? |
This problem is now recorded in #1138.
It was just an example. You can for instance take a 500 kB |
Problem scope
App version
Android version and device/firmware type
Android 14/AP2A.240905.003.F1.ova.2024101100
Steps to reproduce
Actual result
DAVx5 silently stops the synchronization at the event with the big picture. As a result, neither the event with the big picture, nor the events follow it does not arrive to the device.
Expected result
At least DAVx5 should throw an exception, as it did before 4.4.3.2-ose (see the log in the Further info).
Further info
Here is, a patch which cuts large messages. Cutting the messages considered as an alternative to throw an exception, as on the one hand, the patch shows the user that something went wrong, but on the other hand, it allows DAVx5 to finish the synchronization even if it encounters malformed messages.
Here is the log:
The text was updated successfully, but these errors were encountered: