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

Command files:checksums:verify should inform that it is not prepared to work with encryption instead of fail. #31136

Open
SergioBertolinSG opened this issue Apr 16, 2018 · 4 comments

Comments

@SergioBertolinSG
Copy link
Contributor

SergioBertolinSG commented Apr 16, 2018

Steps to reproduce

  1. Having a server with encryption enabled.
  2. Run files:checksums:verify command.

Expected behavior

Inform the admin user running the command that this command is not available when encryption is enabled.

Actual behavior

It tries to run, but it fails because the command is not prepared to run with encryption.

screen shot 2018-04-16 at 12 34 20

Server configuration

Operating system:
Ubuntu 16.04

Web server:
Apache

Database:
MySQL

PHP version:
7.1

Cache
DB

ownCloud version: (see ownCloud admin page)
{"installed":true,"maintenance":false,"needsDbUpgrade":false,"version":"10.0.8.1","versionstring":"10.0.8 RC1","edition":"Enterprise","productname":"ownCloud"}

Updated from an older ownCloud or fresh install:
Fresh

The content of config/config.php:


Are you using external storage, if yes which one: local/smb/sftp/...
No.

Are you using encryption:
Yes.

Logs

{"reqId":"Ugypt0xyy66gMfI4Bvoj","level":3,"time":"2018-04-16T10:32:33+00:00","remoteAddr":"","user":"--","app":"no app in context","method":"--","url":"--","message":"Can not decrypt this file, probably this is a shared file. Please ask the file owner to reshare the file with you."}

Client configuration

Browser
Chrome.

cc @IljaN @pmaier1

@SergioBertolinSG SergioBertolinSG added this to the QA milestone Apr 16, 2018
@ownclouders
Copy link
Contributor

GitMate.io thinks possibly related issues are #3964 (Encryption works only with uploaded files), #12685 (Versioning not working with Encryption), #23786 (files:transfer-ownership command does not work in some scenarios with encryption), #31008 ([stable10] Verify checksums command), and #29081 (decrypt-all command fails to decrypt files when user specific key is enabled in the encryption).

@pmaier1 pmaier1 added the p3-medium Normal priority label Apr 16, 2018
@pmaier1
Copy link
Contributor

pmaier1 commented Apr 16, 2018

Good finding. Considered as nice-to-have from my POV.

@PVince81
Copy link
Contributor

moving to 10.0.9

@PVince81 PVince81 modified the milestones: QA, development, planned Apr 17, 2018
@binarykitchen
Copy link

Having the same problem here and it's a pain.

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

No branches or pull requests

5 participants