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

Encryption 9.1.6 > 10.0.2 default module enabled #28985

Closed
voroyam opened this issue Sep 12, 2017 · 7 comments

Comments

@voroyam
Copy link
Contributor

commented Sep 12, 2017

Steps to reproduce

  1. Install ownCloud 9.1.6
  2. enable default encryption module
  3. Upgrade to 10.0.2

Expected behaviour

User specific key enabled, no drop down menu to choose master key encryption

Actual behaviour

Drop down menu, for users to choose the different encryption modes.

@PVince81 PVince81 added this to the qa milestone Sep 12, 2017
@PVince81

This comment has been minimized.

Copy link
Member

commented Sep 12, 2017

@sharidas please have a look. If any of the modes was already active when upgrading, the oc_appconfig keys need to be set appropriately to make sure the UI doesn't appear any more.

@PVince81

This comment has been minimized.

Copy link
Member

commented Sep 13, 2017

@davitol can you test that the workaround works: basically the admin should select whatever mode they already selected in the past. We'll add an entry in the release notes.

@davitol

This comment has been minimized.

Copy link
Contributor

commented Sep 13, 2017

Scenario 1 - Upgrade to 10.0.2 and do not select any option in the dropdown: Encrypted files are still available.
Scenario 2 - Upgrade to 10.0.2 and select user-key option in the dropdown (same as before): Encrypted files are still available.

Scenario 2 - Upgrade to 10.0.2 and select master key option in the dropdown: Encrypted files are no longer available. (see screenshot)

screen shot 2017-09-13 at 17 49 52

@sharidas

This comment has been minimized.

Copy link
Contributor

commented Sep 29, 2017

Both the changes for:

@PVince81

This comment has been minimized.

Copy link
Member

commented Oct 9, 2017

@sharidas close ?

@sharidas

This comment has been minimized.

Copy link
Contributor

commented Oct 9, 2017

Closing this issue as the changes are merged.

@lock

This comment has been minimized.

Copy link

commented Aug 1, 2019

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.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
5 participants
You can’t perform that action at this time.