CONDSTORE log message insufficiently detailed

[bernd-wechner]

Steps to reproduce

1. This is an observation of log messages, not something to reproduce. Check you logs and if you see this kind of message:

[mail] Error: Sync failed for 7:INBOX: The server does not support the CONDSTORE extension. POST /index.php/apps/mail/api/mailboxes/79/sync

then you have reproduced the issue. If not then not. but The bug is the specificity of the log message (identifying Inbox 7 and not the actual server address - as if I know which one is INBOX 7?)

Expected behavior

If the server does not support the CONDSTORE extension, I hope to see a message in my logs telling me which server that is. And to be honest, what I might be able to do about it.

Actual behavior

The log message does not help me identify which of the many accounts I'm syncing is hosted on a server lacking CONDSTORE., in a way I can understand (identifies as server number 7).

Mail app version

5.5.11

Nextcloud version

31.0.9

Mailserver or service

I don't know, that is the very bug being reported

Operating system

Linux, Ubuntu (Linux 6.8.0-79-generic x86_64)

PHP engine version

Other

Nextcloud memory caching

No response

Web server

Lighttpd

Database

PostgreSQL

Additional info

A fairly top heavy form this for a simple logging bug. Shrug. I'd just find where it's generated in the code base and add to the output a description of the actual server (the servers address)

[ChristophWurst]

Thanks for the report.

The bug form has the minimal infos we typically need to understand a bug. I don't see anything irrelevant.

7 is the account ID. Look into oc_mail_accounts and the row with id=7.

Hope this answers your concerns

[bernd-wechner]

So simply adding some or part of this to the message would help sysadmins some:

SELECT name, email, inbound_host, outbound_host FROM public.oc_mail_accounts WHERE id=7;

Thanks it did help me identify the account, and I'll look into it now (with the provider), but I'd still request that something more useful than the id is shared in the log message. The email field is enough and identifies which account to most any admin I think.

[ChristophWurst]

The email may not be unique. Two users can set up the same account. It happens, rarely.

[nextcloud-command]

Hi there! It looks like this issue hasn’t seen much activity for a while and might need a bit more information to move forward. If we don’t hear back soon, we may close it to keep things organized. Thanks again for your contributions – we really appreciate it!