-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Fatal error after News app update using core updater #18391
Comments
Hi, had noticed the same and using nearly the same setup as the OP except Debian Jessie instead of Arch and nginx instead of Apache. Manually installing the app from apps.owncloud.com did the trick so it seems the archive is ok but not extracted correctly by the app installer. |
@Raydiation what do you think? |
Dup, bug in core ;) #18315 |
@Raydiation Don't think so. #18315 is about old PHP versions where we both have PHP 5.6 installed. Manually installing the app is also working so its not an dependency issue. The app archive is just not extraced completely. |
OK different error it seems |
No idea then, someone needs to audit the installer code and check what goes wrong. |
Well - to be honest - if a file is not there which is expected to be there, it is expected to fall part. Won't fix from my pov - no ide why the file is missing |
Package from apps.owncloud.com and git works just fine, same goes for installing over the web interface. Everything works fine from my POV, haven't tried updating though. |
the file is missing in this list as well |
@anonimal btw you can run ./occ news:verify-install to check which files are missing/outdate, see https://github.com/owncloud/news#commands |
File was missing here during the update from 5.3.9 to 6.0.0 and also from 6.0.0 to 6.0.1 so this wasn't just a network outtage or similar. This also happened only with the news app. Is there anything different in the news app archive compared to apps like calendar and music? |
I don't think there's anything out of the ordinary. |
Ok. Is there anything what i could do to debug this? I have some time in two days to have a closer look at this. |
Try to reproduce it reliably :) |
And post the steps |
@Raydiation running occ on the command line will still throw the same fatal error. $ sudo -u http php occ news:verify-install
PHP Fatal error: require_once(): Failed opening required '/usr/share/webapps/owncloud/apps/news/appinfo/../vendor/autoload.php' (include_path='/usr/sha in /usr/share/webapps/owncloud/apps/news/appinfo/app.php on line 18 Now, the git version: $ sudo -u http php occ news:verify-install
558/558 [============================] 100%
Installation verified, everything OK! |
@anonimal right, I also need a vendor lib for that, that's why the error is thrown. If you've installed composer you could run
inside the news app directory to generate the autoload.php (if the composer.json is there) In any case the install does not validate :) |
Confirmed: |
The verify command should run further after this |
Command returns: PHP Fatal error: Class 'OCA\News\Config\AppConfig' not found in /usr/share/webapps/owncloud/apps/news/appinfo/application.php on line 62 |
Ok, forget it, too much is missing apparently. |
I'm able to reproduce this reliable by following those steps:
There is nothing in the logfiles (even with loglevel 0) expect the ones above. |
I was just trying to reproduce this - autoload.php does exist for me after upgrade - but news is broken due to this:
|
So updating randomly deletes files? Maybe race conditions? We are first extracting the archive, then deleting the old one and move the new folder over, right? |
After reloading the page it worked without issue. |
Just tested with the new release of the Tasks app from 0.7.1 to 0.8.0 which worked without any problem. Few days ago tested the update of the news app from 6.0.1 to 6.0.3 and the same happens. Is there just anything in the archive of the news app which can't be handled by core? Any known bugs of PHP with extracting archives etc? |
ZIP vs tar.gz? |
Could be. I can't remember if the versions 5.3.9 or below where all also tar.gz or what format have those releases used? |
We've used tar.gz version 2.x IIRC |
@RealRancor What distro do you use? Maybe it's a problem with the used libraries. |
@MorrisJobke |
@MorrisJobke I don't think that a failing extraction of an archive will be solved by code signing. This issue here is caused by the tar.bz2 format used by the new app. Other apps using the .zip format are not showing thid problem. |
Oh ... I thought it was because of a broken (partly) extraction. |
Could be fixed by #20196 which is part of 8.1.5, 8.2.1 and 9.0.0 |
@RealRancor Can I ask you to test this again with one of the above versions? |
@MorrisJobke Thanks for the ping, have missed the previous comment. Just tested with oC 8.2.2 on PHP 5.6.14:
When having a look at filesystem level the /apps/news folder contains only the following folders and files:
where the full 6.1.1 from https://apps.owncloud.com/content/show.php/News?content=168040 has this list of files and folders in its main dir:
With 8.2.2 there is also now another error:
|
Just had another look at this because i have seen #22875 has I have absolutely no idea about all that stuff but might worth a look? |
Re-Tested within an updated environment: ownCloud 9.0.1 After hitting the update button to 8.3.0 the archive is still not extracted fully (missing files) and causing an internal server error because of this. Strange enough the News 7.1.2 was pulled/installed successfully to 8.2.1 during the upgrade from ownCloud 8.2.3 to 9.0.1 when running ./occ upgrade. Are there any differences between those two methods? |
Is this still an issue with recent versions ? |
@PVince81 Will re-test once 9.1.1 is out and my productive env is updated to that version |
User has left github and no feedback from OP, closing |
What more was needed of me? |
Most likely #18391 (comment) or any other feedback since your last comment 1 1/2 years ago.
What are you trying to achieve with such comments? |
I haven't used owncloud since. I simply got the ball rolling, I'm not an owncloud developer. |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Steps to reproduce
Expected behaviour
Successfully updated app.
Actual behaviour
Breaks owncloud (blank page of death) because of unfulfilled
require_once()
(files are simply not there).Server configuration
Operating system:
Arch
Web server:
Apache 2.4.16
Database:
MariaDB 10.0.21
PHP version:
5.6.12
ownCloud version: (see ownCloud admin page)
8.1.1
Updated from an older ownCloud or fresh install:
Fresh install
Logs
Web server error log
ownCloud log (data/owncloud.log)
Temporary working solution (via git)
News app files provided by updater
See this paste:
https://zerobin.net/?c73056692ca37283#VY3LeHqDOvk9kOXsIPyMkwlLnA3WUsmQl9ucu7eIxYU=
Notes
I've uninstalled the app and have done a fresh install of the app (instead of updating) and am given the same errors. Cloning directly from the repo works as expected.
The text was updated successfully, but these errors were encountered: