nextcloud / desktop

💻 Desktop sync client for Nextcloud
https://nextcloud.com/install/#install-clients
GNU General Public License v2.0
3.03k stars 797 forks source link

[Bug]: 3.14.0 does not create private.key for E2EE when initializing E2EE #7205

Open bcutter opened 1 month ago

bcutter commented 1 month ago

⚠️ Before submitting, please verify the following: ⚠️

Bug description

After resetting E2EE for a user (starting from scratch) when trying to enable E2EE on the 3.14.0 endpoint, the button "set up encryption" (translated) does nothing. In the back, a Username.public.key is created, while a Username.private.key isn't. That leads to a strange half-setup E2EE with the UI at some point (after restarting the client) allowing to show the seed, where there's not even a box (no seed shown).

Server log spammed with

[end_to_end_encryption] Warnung: Could not find the private key of the user: Username
    GET /ocs/v2.php/apps/end_to_end_encryption/api/v1/private-key
    von xxx.xxx.xxx.xxx von Username um 25.09.2024, 15:00:19

I spent two hours on trying to fix this along with another 3.14.0 specific issue (https://github.com/nextcloud/desktop/issues/7204). E2EE with NC is annoying as hell as it breaks every few weeks or at best months. I'm now at a point where I simply can't trust any NC (endpoint) update anymore. No idea how 3.14.0 could be released with so fundamental issues not fixed (not tested enough?).

No idea how 3.14.0 could be released with so fundamental issues not fixed (not tested enough?).

Solved by downgrading to 3.14.0, like it works for https://github.com/nextcloud/desktop/issues/7171 too.

Steps to reproduce

  1. Reset E2EE on server
  2. Remove E2EE from Windows endpoint running 3.14.0
  3. Try to set up E2EE on the 3.14.0 Windows endpoint - pressing the "enable E2EE" will do nothing

Expected behavior

E2EE can be set up as expected. At least SOME response should be given.

Which files are affected by this bug

none

Operating system

Windows

Which version of the operating system you are running.

Windows 11

Package

Official Windows MSI

Nextcloud Server version

28.0.10

Nextcloud Desktop Client version

3.14.0

Is this bug present after an update or on a fresh install?

Updated to a major version (ex. 3.3.6 to 3.4.0)

Are you using the Nextcloud Server Encryption module?

Encryption is Disabled

Are you using an external user-backend?

Nextcloud Server logs

[end_to_end_encryption] Warnung: Could not find the private key of the user: Username
    GET /ocs/v2.php/apps/end_to_end_encryption/api/v1/private-key
    von xxx.xxx.xxx.xxx von Username um 25.09.2024, 15:00:19

Additional info

Should be mentioned at https://github.com/nextcloud/desktop/issues/7191. Tested with two Windows 11 endpoints, one with normal sync and one with VFS sync, both with same behaviour/output.

mgallien commented 1 month ago

@bcutter Could you test again with the 3.14.1 release ?

bcutter commented 1 month ago

I certainly could. But yesterday when it was released I did not find anything related in the release notes which seems to address this issue.

mgallien commented 1 month ago

@bcutter I have no idea why you have this issue and if that could be related to other solved issues in 3.14.1. You providing such feedback would be very important for us to understand what is happening for you.

bcutter commented 1 month ago

I understand. But I am still completely tired of resetting E2EE as it comes with a lot of work afterwards. And for checking if a new private key is created I need to reset E2EE...