Breaking changes for Nextcloud 32

[susnux]

In Nextcloud 32 the login chain was changed, this means to allow login by email you need to adjust your code. This means checkPassword of the user backend will no longer receive the user id when logged in by email but the email directly.

So to allow to login by email you need to fetch the matching user id of that email manually in your code. You can have a look at the local backend on how to do this: https://github.com/nextcloud/server/blob/3c4feff028ab87b57a29e5771cde8ba6c5b7e4b0/lib/private/User/Database.php

See also: https://github.com/nextcloud/server/pull/47686

[VVD]

Can you suggest the patch?

[stefangweichinger]

So the addon is not working with NC-32 right now, is that correct?

[stefangweichinger]

Dear developers: Please share some information. I have a customer asking ... waiting for NC-32 etc thanks for your work (and infos!)

EDIT: In my case it's IMAP ...

[stefangweichinger]

I set up an extra NC-clone for testing this addon with NC-32 and it works (with the release from gh, not from the NC-app-store). While it's nice to see it working it leaves me uncertain how to proceed, given this issue etc

[beanjammin]

@stefangweichinger I have been looking into alternatives and have had success with OIDC, which my email system and other services also support. I am using the OpenID Connect app to integrate with Nextcloud. I am using keycloak for authentication.

[stefangweichinger]

thanks @beanjammin . In this case I have to either use (Samba) AD or the database behind postfix and dovecot (which I use right now by using this nc addon). I will research OIDC etc