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

ownbackup causing integrity and internal errors on Nextcloud 12 #4765

Closed
joekerna opened this issue May 9, 2017 · 7 comments
Closed

ownbackup causing integrity and internal errors on Nextcloud 12 #4765

joekerna opened this issue May 9, 2017 · 7 comments

Comments

@joekerna
Copy link

joekerna commented May 9, 2017

Steps to reproduce

  1. Activate ownbackup app
  2. click on Additional settings in the admin section

Expected behaviour

The Additional settings section should appear

Actual behaviour

An internal server error appears

Anforderungskennung: 2UwA3TjFmoTXkWO7k9bT

Server configuration

Operating system: Ubuntu 17.04

Web server: Nginx

Database: Mysql

PHP version: 7.1.5

Nextcloud version: 12 Beta 2

Updated from an older Nextcloud/ownCloud or fresh install: Updated

Where did you install Nextcloud from: Web Updater

Signing status:

Signing status
Technical information
=====================
The following list covers which files have failed the integrity check. Please read
the previous linked documentation to learn more about the errors and how to fix
them.

Results
=======
- ownbackup
	- EXCEPTION
		- OC\IntegrityCheck\Exceptions\InvalidSignatureException
		- Certificate is not valid.

Raw output
==========
Array
(
    [ownbackup] => Array
        (
            [EXCEPTION] => Array
                (
                    [class] => OC\IntegrityCheck\Exceptions\InvalidSignatureException
                    [message] => Certificate is not valid.
                )

        )

)

Are you using external storage, if yes which one: Dropbox

Are you using encryption: no

Client configuration

Browser: Firefox 49.9.0 ESR

Operating system: Windows 10

Logs

Nextcloud log (data/nextcloud.log)

Nextcloud log
Exception: The requested uri(/settings/admin/ocs/v2.php/apps/notifications/api/v2/notifications) cannot be processed by the script '/apps/notifica')

    /var/www/nextcloud/lib/base.php - line 927: OC\AppFramework\Http\Request->getRawPathInfo()
    /var/www/nextcloud/index.php - line 40: OC handleRequest()
    {main}



OCP\AppFramework\QueryException: Could not resolve AppName! Class AppName does not exist

    /var/www/nextcloud/lib/private/AppFramework/Utility/SimpleContainer.php - line 117: OC\AppFramework\Utility\SimpleContainer->resolve('AppName')
    /var/www/nextcloud/lib/private/ServerContainer.php - line 116: OC\AppFramework\Utility\SimpleContainer->query('AppName')
    /var/www/nextcloud/lib/private/AppFramework/Utility/SimpleContainer.php - line 66: OC\ServerContainer->query('AppName')
    /var/www/nextcloud/lib/private/AppFramework/Utility/SimpleContainer.php - line 96: OC\AppFramework\Utility\SimpleContainer->buildClass(Object(ReflectionClass))
    /var/www/nextcloud/lib/private/AppFramework/Utility/SimpleContainer.php - line 117: OC\AppFramework\Utility\SimpleContainer->resolve('OCA\\OwnBackup\\C...')
    /var/www/nextcloud/lib/private/ServerContainer.php - line 116: OC\AppFramework\Utility\SimpleContainer->query('OCA\\OwnBackup\\C...')
    /var/www/nextcloud/lib/private/AppFramework/DependencyInjection/DIContainer.php - line 410: OC\ServerContainer->query('OCA\\OwnBackup\\C...')
    /var/www/nextcloud/apps/ownbackup/admin.php - line 19: OC\AppFramework\DependencyInjection\DIContainer->query('\\OCA\\OwnBackup\\...')
    /var/www/nextcloud/lib/private/legacy/app.php - line 727: include('/var/www/nextcl...')
    /var/www/nextcloud/settings/Controller/AdminSettingsController.php - line 102: OC_App getForms('admin')
    /var/www/nextcloud/settings/Controller/AdminSettingsController.php - line 93: OC\Settings\Controller\AdminSettingsController->getLegacyForms()
    /var/www/nextcloud/settings/Controller/AdminSettingsController.php - line 73: OC\Settings\Controller\AdminSettingsController->getSettings('additional')
    [internal function] OC\Settings\Controller\AdminSettingsController->index('additional')
    /var/www/nextcloud/lib/private/AppFramework/Http/Dispatcher.php - line 160: call_user_func_array(Array, Array)
    /var/www/nextcloud/lib/private/AppFramework/Http/Dispatcher.php - line 90: OC\AppFramework\Http\Dispatcher->executeController(Object(OC\Settings\Controller\AdminSettingsController), 'index')
    /var/www/nextcloud/lib/private/AppFramework/App.php - line 114: OC\AppFramework\Http\Dispatcher->dispatch(Object(OC\Settings\Controller\AdminSettingsController), 'index')
    /var/www/nextcloud/lib/private/AppFramework/Routing/RouteActionHandler.php - line 47: OC\AppFramework\App main('OC\\Settings\\Con...', 'index', Object(OC\AppFramework\DependencyInjection\DIContainer), Array)
    [internal function] OC\AppFramework\Routing\RouteActionHandler->__invoke(Array)
    /var/www/nextcloud/lib/private/Route/Router.php - line 299: call_user_func(Object(OC\AppFramework\Routing\RouteActionHandler), Array)
    /var/www/nextcloud/lib/base.php - line 975: OC\Route\Router->match('/settings/admin...')
    /var/www/nextcloud/index.php - line 40: OC handleRequest()
    {main}

This is in reference to pbek/nextbackup#18:

Activating ownbackup on Nextcloud 12 causes integrity errors (see log above) and it also causes the Additional settings section to fail.

Deactivating the app enables the Additional settings section again, the integrity errors remain.

@JSoko
Copy link
Member

JSoko commented May 9, 2017

Had that issue too

  • Just go to apps
  • deactivate it
  • deinstall it
  • install and activate it
  • run the integry-check again
    fixed it for me...

@MorrisJobke
Copy link
Member

You need to install the app from the App Store to have a proper signature. Installing the git version is usually not working, because once a file changes the signature is invalid.

If I understand it correctly this problem was when installing the app via a github zip file.

I wil close this, if this was the App Store install, then we are happy to look into this again.

Thanks

@Laupe
Copy link

Laupe commented May 9, 2017

@MorrisJobke This error also occurs when downloading from App Store.

root@private:/home/cloud_sudo -u www-data php /home/apps/cloud/occ integrity:check-app qownnotesapi
  - EXCEPTION:
    - class: OC\IntegrityCheck\Exceptions\InvalidSignatureException
    - message: Certificate is not valid.

pbek/qownnotesapi#12

I have a problem with multiple applications.

I have done the exact procedure described by @JSoko

Log

{"reqId":"nh0jEqv0Bp2aIcLe828j","level":3,"time":"2017-05-09T17:37:29+00:00","remoteAddr":"","user":"--","app":"PHP","method":"--","url":"--","message":"Undefined index: admin at \/home\/apps\/cloud\/apps\/news\/lib\/Config\/AppConfig.php#91","userAgent":"--","version":"12.0.0.16"}

I have the same problem here. Also downloaded from the App Store nextcloud/external#23

Even it happened to me that the file is not at all signed if I download it from the store. The archive is missing signature.json
image

App store returns badly signed archives pbek/qownnotesapi#12 (comment) (#issuecomment-300246174)

@joekerna
Copy link
Author

joekerna commented May 9, 2017

@MorrisJobke you misunderstood me. I did download the app from the appstore. I only went to github to create the issue because I thought it was an issue with the app itself.
I tried @JSoko 's suggestion but without success.

I think you should reopen the issue.
Thanks

@MorrisJobke MorrisJobke reopened this May 9, 2017
@nursoda
Copy link

nursoda commented May 11, 2017

This bug addresses TWO issues:

  • signature/integrity of ownbackup
  • app or signature causing additional settings section to crash
    I think they should be separated.

Concerning the integrity, there is another bug (in ownbackup issues) already: pbek/nextbackup#10
Concerning the crash, I just added another bug (duplicate?) since I did not deduct from the title of this bug that it is related to admin settings: #4823. It is caused by ownbackup, and will be fixed once pbek/nextbackup#17 is released.

@MorrisJobke
Copy link
Member

since I did not deduct from the title of this bug that it is related to admin settings: #4823

Then close this one here and go for #4823

@pbek
Copy link
Member

pbek commented May 12, 2017

The settings crash was handled in pbek/nextbackup#17 but the InvalidSignatureException is still there. It's the same with https://github.com/pbek/qownnotesapi/. Both apps are in the NC store, were signed with a ownCloud certificate and did work until NC 11.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants