-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Theming app (still) not working in NC 12.0.3 #6581
Comments
cc @nextcloud/theming |
Follow-up issue to #6267. |
Can you check your php settings, if that file may not be uploaded because of size limitations in
It seems we don't catch errors within the $_FILES variable when uploading a logo, let me check out if we can at least provide proper error messages if something goes wrong there. |
This is my php.ini, which I merged with the one found for the docker image. cat php.ini | grep -vE "^.(;|#)." | grep -vE "^$"
|
Here is my "merged" nginx config:
|
BTW: Everything else in NC with all these apps is working perfectly. Photos, Videos, Documents, WebRTC-stuff. Really everything else works. |
@croessner Thanks for the detailed information. Although I cannot find any issue in that right now. |
@juliushaertl I will give it a try in the next hour and report back... |
@juliushaertl 👍 You rock!! Your patches finally fixed my problems! Even though I had to create the dirs tests/Controller first. But it works. So happy. Thanks a lot |
@croessner Glad i could help. Tests are not included in the release packages, as they are not required there. Just out of interest, was there an issue with the file you uploaded? |
@juliushaertl there seems to be one bug remaining :-( If I have set the background login image for the first time with your patched version, I can neither reset, restore nor set a new image. It always says that it saved it successfully. Nothing is logged in nextcloud.log nor in the JavaScrip console. It simply does not do the job. It keeps the current image |
I have the same issue in my Nextcloud-snap 12/Edge installation. Obviously I can't change anything in the snap. |
Seems to be some browser caching issue. Is it possible to set some headers for the theming app that prevents caching? |
@croessner I cannot reproduce the issue. Restoring/setting a new image works fine on my setup. |
Btw. we have a cachebuster on every resource, so after changing a value in the theming app all files that depend on the theming should be loaded again. |
@croessner , it doesn't related to any browser at all. The issue been reproduced with all browsers in different systems. I have a lot of browsers that work in private mode with deleting all cache/history/cookies, the issue is there as well. |
Same problem. Color changes in the iPad app, not in the browser. |
We improved the caching logic of the theming app recently to better invalidate existing themes. Could you try again with 13.0.4? This should be solved in there. I will close this ticket for now. If this happens again, please ping here and we can then reopen the ticket. |
Steps to reproduce
Expected behaviour
Logo should be uploaded and set
Actual behaviour
Dialog says that it stored the image and I see a broken link in the preview. Of logging out and back in there is still only the the default NC logo.
Server configuration
Operating system:
Gentoo Linux
Web server:
Installed versions: 1.13.5(mainline)^t(06:34:23 PM 09/19/2017)(aio http http-cache http2 ipv6 pcre ssl threads vim-syntax -debug -libatomic -libressl -luajit -pcre-jit -rtmp -selinux NGINX_MODULES_HTTP="access auth_basic autoindex browser cache_purge charset dav dav_ext empty_gif fancyindex fastcgi geo gzip gzip_static headers_more image_filter limit_conn limit_req map memcached mp4 proxy push_stream realip referer rewrite scgi spdy split_clients ssi upstream_ip_hash userid uwsgi xslt -addition -auth_ldap -auth_pam -auth_request -degradation -echo -flv -geoip -gunzip -lua -memc -metrics -mirror -mogilefs -naxsi -perl -random_index -secure_link -security -slice -slowfs_cache -sticky -stub_status -sub -upload_progress -upstream_check -upstream_hash -upstream_keepalive -upstream_least_conn -upstream_zone" NGINX_MODULES_MAIL="-imap -pop3 -smtp" NGINX_MODULES_STREAM="-access -geo -geoip -limit_conn -map -realip -return -split_clients -ssl_preread -upstream_hash -upstream_least_conn -upstream_zone" USERLAND="GNU")
Database:
Installed versions: 10.1.26{tbz2}(11:40:56 AM 09/11/2017)(backup bindist cracklib extraengine innodb-lz4 innodb-lzo innodb-snappy jemalloc openssl pam perl server -debug -embedded -galera -jdbc -kerberos -latin1 -libressl -mroonga -odbc -oqgraph -profiling -selinux -sphinx -sst-rsync -sst-xtrabackup -static -static-libs -systemd -systemtap -tcmalloc -test -tokudb -xml -yassl ABI_MIPS="-n32 -n64 -o32" ABI_PPC="-32 -64" ABI_S390="-32 -64" ABI_X86="64 -32 -x32" ELIBC="-FreeBSD")
PHP version:
Installed versions: 7.0.23(7.0)(06:32:03 PM 09/19/2017)(acl bcmath berkdb bzip2 calendar cgi cli crypt ctype curl exif fileinfo filter flatfile fpm ftp gd gdbm gmp hash iconv imap inifile intl iodbc ipv6 json ldap ldap-sasl mhash mysql mysqli nls opcache pcntl pdo phar posix postgres readline session simplexml snmp soap sockets spell sqlite ssl sysvipc threads tidy tokenizer truetype unicode wddx xml xmlreader xmlrpc xmlwriter xpm xslt zip zlib -apache2 -cdb -cjk -coverage -debug -embed -enchant -firebird -kerberos -libedit -libressl -mssql -oci8-instant-client -odbc -phpdbg -qdbm -recode -selinux -sharedmem -systemd -webp)
Nextcloud version: (see Nextcloud admin page)
12.0.3
Updated from an older Nextcloud/ownCloud or fresh install:
Updated from 12.0.3RC2
Where did you install Nextcloud from:
Official website
Signing status:
No errors have been found.
List of activated apps:
Nextcloud configuration:
Are you using external storage, if yes which one: local/smb/sftp/...
Dropbox and another NC12 instance
Are you using encryption: yes/no
Module loaded, but not used.
Are you using an external user-backend, if yes which one: LDAP/ActiveDirectory/Webdav/...
LDAP
LDAP configuration (delete this part if not used)
Client configuration
Browser:
Safari-11.0
Operating system:
macOS Sierra
Logs
Web server error log
Nextcloud log (data/nextcloud.log)
Browser log
The text was updated successfully, but these errors were encountered: