Skip to content
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

[Bug]: Contacts in NC 25 have broken avatar images; user avatars work fine. #35582

Closed
7 of 9 tasks
summersab opened this issue Dec 4, 2022 · 1 comment
Closed
7 of 9 tasks
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 25-feedback bug

Comments

@summersab
Copy link
Contributor

summersab commented Dec 4, 2022

⚠️ This issue respects the following points: ⚠️

  • This is a bug, not a question or a configuration/webserver/proxy issue.
  • This issue is not already reported on Github (I've searched it).
  • Nextcloud Server is up to date. See Maintenance and Release Schedule for supported versions.
  • Nextcloud Server is running on 64bit capable CPU, PHP and OS.
  • I agree to follow Nextcloud's Code of Conduct.

Bug description

This is a copy of a bug filed over on the contacts app:
nextcloud/contacts#3078

My recent comment on the issue is as follows:

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:

throw new \Exception('user does not exist');

None of the libraries in the contacts app are even accessed to trigger this bug, so I am pretty certain it has to do with the core.

The contents of the original bug report is below:

I noticed 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 placeholder. In the following screenshot the "C" and "D" are users in Nextcloud. All other entries are just address book contacts without imageimage

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.

Installation method

Community Manual installation with Archive

Operating system

Debian/Ubuntu

PHP engine version

PHP 8.1

Web server

Apache (supported)

Database engine version

MariaDB

Is this bug present after an update or on a fresh install?

Fresh Nextcloud Server install

Are you using the Nextcloud Server Encryption module?

Encryption is Disabled

What user-backends are you using?

Configuration report

No response

List of activated Apps

Enabled:
  - cloud_federation_api: 1.8.0
  - contacts: 5.0.1
  - dav: 1.24.0
  - federatedfilesharing: 1.15.0
  - files: 1.20.1
  - lookup_server_connector: 1.13.0
  - oauth2: 1.13.0
  - password_policy: 1.15.0
  - provisioning_api: 1.15.0
  - settings: 1.7.0
  - theming: 2.0.1
  - twofactor_backupcodes: 1.14.0
  - viewer: 1.9.0
  - workflowengine: 2.7.0

Nextcloud Signing status

No response

Nextcloud Logs

No response

Additional info

No response

@summersab summersab added 0. Needs triage Pending check for reproducibility or if it fits our roadmap bug labels Dec 4, 2022
@joshtrichards
Copy link
Member

Fixed in #36735

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 25-feedback bug
Projects
None yet
Development

No branches or pull requests

3 participants