-
Notifications
You must be signed in to change notification settings - Fork 173
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
Contacts initiates a new request for each contact picture - when scrolling, large number of php processes #2420
Comments
An addition suggestion, if a contact has no picture, potentially (maybe a setting option), assign the coloured circle to the contact, so the loading issue only occurs on first contact display. |
In addition I get a 404 error in the webserver log for each contact that has no profile picture. |
Can confirm this issue for Nextcloud 24.0.1 and Contacts version 4.1.1. When opening the contacts app the requests made to load the Avatar images bring down the instance :( |
I managed to work around this issue by adding a rule to "My Filters" in the uBlock origin settings:
The downside of course is that you do not have any contact images any more 😉 |
Hi, Operating system: Linux 3.10.0-1160.31.1.el7.x86_64 #1 SMP Thu Jun 10 13:32:12 UTC 2021 x86_64 Webserver: Apache/2.4.54 (Unix) (fpm-fcgi) Database: mysql 10.3.22
Modules loaded: Core, date, libxml, openssl, pcre, zlib, filter, hash, json, Reflection, SPL, session, standard, cgi-fcgi, restrictdocroot, bcmath, bz2, calendar, ctype, curl, dba, dom, mbstring, fileinfo, ftp, gd, gettext, gmp, iconv, imap, intl, ldap, exif, mysqlnd, PDO, pgsql, Phar, posix, shmop, SimpleXML, soap, sockets, sodium, sqlite3, sysvmsg, sysvsem, sysvshm, tidy, tokenizer, xml, xmlwriter, xsl, mcrypt, mysqli, pdo_mysql, pdo_pgsql, pdo_sqlite, xmlreader, xmlrpc, zip, apcu, decimal, gnupg, igbinary, imagick, mailparse, msgpack, yaml, mongodb, redis, Zend OPcache @lyallp can I help with any information needed to triage? @astehlik thanks for your ABP workaround, saves a lot of nerves atm. |
There is also another fix / workaround for this issue, see: nextcloud/server#33048 When the 404 errors are handled correctly and do not cause a redirect to the dashboard the number of requests for loading the contact images should not be a problem any more. |
My argument is that we should never reach the stage of having to deal with 404 in the case of Contact images. |
Is this still being worked on? My contacts list is slow as hell ;( |
Couldn't the cache images for contact initials simply be cached somewhere? |
Steps to reproduce
Contacts with pictures seem to load immediately.
I have hundreds of contacts, of which, only a few actually have pictures associated with them.
Expected behavior
Given I suspect it's the colourful circles with the contact initials that is being generated server side is causing the issue.
Maybe only generate 5 (an arbitrary number) concurrently, rather than simply blasting out potentially hundreds of requests, if the end user scrolls quickly.
Actual behavior
See To Reproduce.
Server configuration detail
Operating system: Linux 2.6.32-954.3.5.lve1.4.78.el6.x86_64 #1 SMP Thu Mar 26 08:20:27 EDT 2020 x86_64
Webserver: LiteSpeed (litespeed)
Database: mysql 10.2.39
PHP version:
7.4.22
Modules loaded: Core, date, libxml, openssl, pcre, sqlite3, zlib, bz2, calendar, ctype, curl, hash, filter, ftp, gettext, gmp, SPL, iconv, pcntl, readline, Reflection, session, standard, shmop, SimpleXML, mbstring, tokenizer, xml, litespeed, bcmath, dom, fileinfo, gd, imagick, imap, intl, json, exif, mysqli, mysqlnd, PDO, pdo_mysql, pdo_sqlite, Phar, posix, soap, sockets, timezonedb, xmlreader, xmlrpc, xmlwriter, xsl, zip, Zend OPcache
Nextcloud version: 22.1.1 - 22.1.1.2
Updated from an older Nextcloud/ownCloud or fresh install: Updated from older version
Where did you install Nextcloud from: downloaded nextcloud source distribution
Signing status
Array
(
)
List of activated apps
Configuration (config/config.php)
Are you using external storage, if yes which one: local
Are you using encryption:
Are you using an external user-backend, if yes which one: local
Client configuration
Browser: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/92.0.4515.159 Safari/537.36
Operating system: Gentoo Linux 64 bit
Logs
Web server error log
Nextcloud log
Browser log
n/a
The text was updated successfully, but these errors were encountered: