-
Notifications
You must be signed in to change notification settings - Fork 84
Implement IProvideUserSecretBackend compatibility for per-user encryption - #697
Implement IProvideUserSecretBackend compatibility for per-user encryption #697summersab wants to merge 1 commit into
IProvideUserSecretBackend compatibility for per-user encryption #697Conversation
2835706 to
cd0c669
Compare
summersab
commented
Feb 24, 2023
@blizzz, you look like one of the top contributors to this repo. Is the CI check broken?
blizzz
commented
May 12, 2023
@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.
immerda
commented
May 20, 2023
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...
cd0c669 to
d5e4825
Compare
d5e4825 to
17b1210
Compare
becfef3 to
f5de408
Compare
f5de408 to
128a9d3
Compare
mglants
commented
Nov 2, 2023
Any news, when it will be supported, readlly want to use saml for server-side encryption
128a9d3 to
6e1548a
Compare
@blizzz
blizzz
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your contribution! Rebased (CI is repaired since) and left a first review.
blizzz
commented
May 2, 2025
@summersab @immerda is this still a thing?
summersab
commented
May 3, 2025
I mean... it should have been. I had to shift away from my NC plans, but this is a feature that provides some advanced encryption capabilities. I wish it would have been merged when I was more active with the project to be honest.
blizzz
commented
May 6, 2025
Understandably 😓
6e1548a to
3112f62
Compare
3112f62 to
0cf1aca
Compare
CarlSchwan
commented
Jun 1, 2026
btw this is now rebased and cleaned up. @summersab if you still have your test setup for this, it would be great if you could give it a try and confirm that it still works
0cf1aca to
d27578f
Compare
Signed-off-by: summersab <18727110+summersab@users.noreply.github.com> Signed-off-by: Andrew Summers <18727110+summersab@users.noreply.github.com> Signed-off-by: Carl Schwan <carlschwan@kde.org>
d27578f to
371646d
Compare
CarlSchwan
commented
Jun 1, 2026
Actually this will need nextcloud/server#60898 otherwise there is no real way to make this feature optional. Now the logic is as following:
Feature enabled: returns either a non-empty-string or throws a HintException
Feature disabled: returns null
Uh oh!
There was an error while loading. Please reload this page.
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
IProvideUserSecretBackendclass 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.