Large battery drain while in background

[Thurion]

Expected behaviour

Not to use much battery when not using the phone at all.

Actual behaviour

Extremely high battery drain when not using the phone.
The iOS battery info shows that the Nextcloud app ran in the background and also drained the battery a lot (Scroll to end of the post for a screenshot).
Auto upload and background app refresh are turned off but I think I enabled the iOS files integration.
Might be related to #1005

Steps to reproduce

Unknown but this happened on my iPhone and iPad. This might help:
Apache access log shows 139,222 entries (counted at 23.30) originating from my IP with this text in one day:

"DELETE /nextcloud/ocs/v2.php/apps/notifications/api/v2/notifications/39 HTTP/1.1" 200 5346 "-" "Mozilla/5.0 (iOS) Nextcloud-iOS/2.25.0"

And the Nextcloud log also shows this several times. Database max connections are set to 100.

[PHP] Error: Doctrine\DBAL\DBALException: Failed to connect to the database: An exception occurred in driver: SQLSTATE[08006] [7] FATAL:  remaining connection slots are reserved for non-replication superuser connections at /var/www/nextcloud/lib/private/DB/Connection.php#64

DELETE /nextcloud/ocs/v2.php/apps/notifications/api/v2/notifications/39
from [IP WITHHELD] at 2019-12-01T22:20:59+00:00

iOS version

13.2.3

App version

2.25.0

Server configuration

Operating system:
Debian 10
Web server:
2.4.38
Database:
PostgreSQL 11.6
PHP version:
7.2
Nextcloud version: (see Nextcloud admin page)
17.0.1

The huge spike at 3-4 a.m. is also Nextcloud.

[marinofaggiana]

Hi @Thurion very, strange ... can you try with the last version in TestFlight 2.25.2 build with the last SDK ?

[Thurion]

I could do that. I never used TestFlight before and it seems that I need an invite code.

[marinofaggiana]

https://testflight.apple.com/join/GjNbfo2a

[j-tai]

Happening to me as well. The following line is repeated 10,014 times in my access log, once every ~1-4 seconds, in the span of about 5 hours:

"DELETE /ocs/v2.php/apps/notifications/api/v2/notifications/18 HTTP/1.1" 200 116 "-" "Mozilla/5.0 (iOS) Nextcloud-iOS/2.25.1"

The battery began to drain after I received a notification from the app—specifically, when someone called me on Nextcloud Talk.

Killing the app (swipe up on app switcher) temporarily stops the battery drain.

[Thurion]

I installed the beta version through TestFlight.
But I can't tell if the battery drain bug is gone or not because the app crashes as soon as I receive a push notification. Aside from that the app seems to behave as expected.

[marinofaggiana]

@Ivansss please look here :-)

[Ivansss]

@marinofaggiana Did you build the TestFlight version with iOS 13 SDK?

[Ivansss]

Regarding the problem with the delete push notifications, we are working on a fix for that.
@marinofaggiana I will create a PR for the changes needed in the iOS app.

[nickvergessen]

@Thurion you could try to apply the following patch on your Nextcloud server:
https://github.com/nextcloud/notifications/pull/505/files

It solves 1 part of the problem and will hide this effect.
It will still do 1 API request too much, but we will solve this in the iOS app.

[marinofaggiana]

@marinofaggiana Did you build the TestFlight version with iOS 13 SDK?

yes, @Ivansss , it's already in TestFlight (2.25.2) XCode 11.2.1

[Ivansss]

yes, @Ivansss , it's already in TestFlight (2.25.2) XCode 11.2.1

Ok. That explains

But I can't tell if the battery drain bug is gone or not because the app crashes as soon as I receive a push notification. Aside from that the app seems to behave as expected.

The app will be shutted down by the operating system when receiving a push notification until we change the way we send push notifications.
I will work on that @marinofaggiana

[Thurion]

@nickvergessen I'd rather not apply patches outside the regular versions on a live server.

[nickvergessen]

It's the only way to prevent the battery drain, until a new iOS version is released and installed, or the same for the notifications app, which will be ~19.12.