user_saml icon indicating copy to clipboard operation
user_saml copied to clipboard
verified publisher icon nextcloud

Implement `IProvideUserSecretBackend` compatibility for per-user encryption

Open summersab opened this issue 3 years ago • 7 comments

This PR is intended to implement the functionality from PR #537. The upstream code changed enough that it was easier to close the original PR and submit a new one.

Now that the IProvideUserSecretBackend class has been added to the Nextcloud core with PR nextcloud/server#24837 / nextcloud/server#27929, this PR adds the necessary logic to support per-user SAML provided secrets.

summersab avatar Feb 10 '23 00:02 summersab

@blizzz, you look like one of the top contributors to this repo. Is the CI check broken?

summersab avatar Feb 24 '23 17:02 summersab

@blizzz, you look like one of the top contributors to this repo. Is the CI check broken?

Only partly, it should be green up to and including stable25. We're working on getting it back for 26 and master.

blizzz avatar May 12 '23 14:05 blizzz

Hey, this is just our yearly reminder that we are still testing this patch in our instance. In fact I just applied this new PR to the newest released nextcloud version and it works perfectly well and is stable enough that we can still decrypt the data from 2.5 years ago when we initially proposed this patch :) We are still eagerly awaiting upstream adoption...

immerda avatar May 20 '23 20:05 immerda

Any news, when it will be supported, readlly want to use saml for server-side encryption

mglants avatar Nov 02 '23 01:11 mglants