Desktop: Password not saved for new accounts (windows, daily builds)

[klada]

When setting up a new account the sync password is not saved in the Windows Credential Store. This means the user needs to reauthenticate after closing and re-opening the sync client.

I suspect this has something to do with the recent upgrade of Qt/QtKeychain, maybe a wrong build option or something like that. I don't think that it has something to do with the QtKeychain 0.9.1 release in general (see #569), as I can confirm that 0.9.1 itself is clean (I compiled a testclient against 0.9.1 and it is working properly).

I could look into this a bit deeper, but I don't have access to the exact build environment bootstrapping mechanism. I am sure you manage the environment dependencies through a platform management tool (Ansible/Chef/Puppet or whatever), would you mind sharing the environment recipe so people can reproduce the environment for debugging?

Steps to reproduce:

1. Quit Nextcloud, remove current config and sync folder
2. Remove all Nextcloud credentials from the Windows Credential Manager
3. Open Nextcloud client (daily build) and set up account
4. Quit Nextcloud client
5. Re-open Nextcloud client → password is lost

[camilasan]

Hi @klada
I am getting to the same conclusion about this bug but I couldn't yet figure out the problem.
Here is the scripts we are using for the builds: https://github.com/nextcloud/client-building
Thanks!

[camilasan]

Hi! I think I solved it!Hi! Could you try this build - please make sure to have a clean configuration - https://download.nextcloud.com/desktop/daily/Windows/Nextcloud-2.5.0.61352-daily-20180904.exe?

Thanks!

[klada]

Very nice, this build is saving the password properly again and also includes the latest QtKeychain version. Thank you.

[camilasan]

The last few commits from https://github.com/nextcloud/client-building fixed the problem.