Broken image display for address book contacts in drop down menu

[arnowelzel]

Describe the bug

I notized this issue after upgrading from Nextcloud 24 to Nextcloud 25, but maybe this also happens with Nextcloud 24:

When opening the contact menu, images for contacts who don't have an image show as "broken image" and not using the initials as placolder. In the following screenshot the "C" and "D" are users in Nextcloud. All other entries are just address book contacts without image

The reason for this is, that the respective image URLs (e.g. https://myserver.example/avatar/a5db8598-935c-4935-8ac7-02d89f1226e6/64&size=64) result in 404 errors.

Steps to reproduce

1. Make sure that the Contacts app is installed and active
2. Create some address book entries without image
3. Click on the "contacts" icon on the top right corner to display the contact drop down menu

Expected behavior

Address book contacts without image show a placholder similar to user entries based on the first and last name.

Actual behavior

A "broken image" is displayed since the image URL responds with a 404 error.

Contact version

5.0.1

Operating system

Ubuntu Linux 20.04.5 LTS

PHP engine version

PHP 8.1

Web server

Apache (supported)

Database

MariaDB

Additional info

No response

[plutocrat]

Can confirm. No contact images displaying anywhere in Nextcloud web app. Not in dropdown, not in contacts app.
Identical synced contacts display fine in Thunderbird / Cardbook.
PHP 8.0, Nextcloud versions 24 and also 25. Ubuntu 20.04. Was previously working but I don't use the web interface much, so couldn't say exactly when it broke.

[nicokaiser]

In the contact app, the 404 errors are also present (seems like the app makes HTTP request for each and every contact to check if there is an image), but a placeholder image is displayed correctly.
This is broken in the dropdown menu.

[arnowelzel]

Well - generating dozens or hundreds of requests which end in 404 errors is not a good design. If the contact app wants to load an image for a contact, then this should either be the stored image or a generated placeholder similar to how it is done for users. But the way how this seems to be implemented at the moment is really not very good.

Having many 404 errors also triggers alerts in the server monitoring and it is hard to see if there is a real problem with the server or just the odd behaviour of the contacts app.

[SKB-CGN]

Same problem here. Even https://server.com/apps/contacts/css/icons.css?v=19bbdecf-15](https://cloud.kreyenborg.koeln/apps/contacts/css/icons.css?v=19bbdecf-15)" wurde wegen eines MIME-Typ-Konfliktes ("text/html") blockiert (X-Content-Type-Options: nosniff).

[dneuhaeuser]

I also have this problem with contacts of federated servers.

The avatar is trying to load from my local nextcloud installation instead of the federated installation:
https://cloud.myserver.de/index.php/avatar/christian......./64&size=32

should be:
https://cloud.myfederation.de/index.php/avatar/christian......./64&size=32

[plutocrat]

Just tried something new: Found a contact without an image in the NC web interface. Added Image, which didn't display anywhere in contacts app or dropdown. Synched contact to phone and email client, where the image displays correctly.

Lots of errors in this format as previously stated.
https://myserver.example/avatar/a5db8598-935c-4935-8ac7-02d89f1226e6/64&size=64

[summersab]

I can confirm that this is a bug. I just spun up a fresh instance of NC with a SQLite DB, and any new contact has a broken image while users have the default avatars with their initials. While tracing the code, I actually think that this issue has to do with the core user libraries. The failure occurs here:
https://github.com/nextcloud/server/blob/ec5cbdeb7ffb87c0169c39e6f44846e819b41f14/lib/private/Avatar/AvatarManager.php#L114

I'm going to open an issue in the server repo referencing this.

[arnowelzel]

I can confirm that this is a bug. I just spun up a fresh instance of NC with a SQLite DB, and any new contact has a broken image while users have the default avatars with their initials. While tracing the code, I actually think that this issue has to do with the core user libraries. The failure occurs here: https://github.com/nextcloud/server/blob/ec5cbdeb7ffb87c0169c39e6f44846e819b41f14/lib/private/Avatar/AvatarManager.php#L114

I'm going to open an issue in the server repo referencing this.

Thanks for that finding! With this information it should not be too hard to fix this.

[summersab]

I submitted a PR to hopefully fix the issue. Until then, here is a very simple app that solves the problem temporarily:
https://github.com/nunimbus/contacts_avatar_patch