I tried to set ‘log_rotate_size’ and now I have to inifinite log warnings (after deleting it)

ℹ️ Support
,
1

[/details]

Nextcloud version: 30.0.0.14
Operating system and version : Debian 12
Apache or nginx version: Apache2
PHP version: PHP8.2.20

I was trying to add rotation to the log system using ‘log_rotate_size’. After I add it to config.php, the warning started to be displayed:

[no app in context] Warning: Link of notification is not an absolute URL and does not work in mobile and desktop clients [app: survey_client, subject: updated]
GET /ocs/v2.php/apps/notifications/api/v2/notifications
from IP_REMOTE by MY_ADMIN_NAME at 21 Sept 2024, 14:10:20

[no app in context] Warning: Icon of notification is not an absolute URL and does not work in mobile and desktop clients [app: survey_client, subject: updated]
GET /ocs/v2.php/apps/notifications/api/v2/notifications
from IP_REMOTE by MY_ADMIN_NAME at 21 Sept 2024, 14:10:20

So I decided to remove the line from the config.php, but the problem persist. I don’t know if there’s something wrong there. In another post I found a reply that mentioned that the config file may be the problem, but I don’t know what it can be.

The log system is displaying this every 30 seconds.

Is this the first time you’ve seen this error? Y

Edit1: When I disable survey_client the warning stops!

2

I think the timing of the appearance of this error was just a coincidence. I don’t think log_rotate_size has any connection to it.

Is this a brand new deployment?

To start, confirm that your overwrite.cli.url looks reasonable (e.g. https://domain.tld/): Configuration Parameters — Nextcloud latest Administration Manual latest documentation

3

Yes, it’s a new deployment, but I started a week ago, and the problem appear today after the mentioned modification.

My overwrite.cli.url looks like:
‘overwrite.cli.url’ => ‘https://cloud.mydomain.com’,

4

Follow-up: this is from some new logging to detect existing problems with notifications from some apps. Fix pending for survey_client. Issue you can subscribe to if you like to track progress: v30 regression: `Link of notification is not an absolute URL and does not work in mobile and desktop clients` · Issue #316 · nextcloud/survey_client · GitHub

If you want to test the proposed fix, you can find it linked from that issue or here: fix: notifications link for mobile / desktop by joshtrichards · Pull Request #317 · nextcloud/survey_client · GitHub

Instructions of patching: Patching Nextcloud — Nextcloud latest Administration Manual latest documentation

5

I’ve seen that it’s fixed and queued for NC30.0.1.

I’ll apply the patch.

Thanks a lot!

6

This topic was automatically closed 90 days after the last reply. New replies are no longer allowed.