-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
AppEcosystem Authentification #39655
Conversation
Signed-off-by: Alexander Piskun <bigcat88@icloud.com>
As talked in the call, i think the cleanest approach would be be to have a AuthenticationProvider interface that apps can implement an register their authentication services then, however since there are plans to move this to OCP anyways it might be ok for now to:
|
Signed-off-by: Andrey Borysenko <andrey18106x@gmail.com>
/backport to stable27 |
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.
👍
The backport to stable27 failed. Please do this backport manually. # Switch to the target branch and update it
git checkout stable27
git pull origin stable27
# Create the new backport branch
git checkout -b fix/foo-stable27
# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts. Resolve them.
git cherry-pick abc123
# Push the cherry pick commit to the remote repository and open a pull request
git push origin fix/foo-stable27 More info at https://docs.nextcloud.com/server/latest/developer_manual/getting_started/development_process.html#manual-backport |
As this was merged, we do not need Authentication patch anymore for Nextcloud core. nextcloud/server/pull/39655 --------- Signed-off-by: Alexander Piskun <bigcat88@icloud.com>
As this was merged, we do not need Authentication patch anymore for Nextcloud core. nextcloud/server#39655 --------- Signed-off-by: Alexander Piskun <bigcat88@icloud.com>
Summary
It's a draft for tomorrow's AppEcosystem call.
This is all that is needed (minimum changes) to authenticate applications that will be written for the new Appecosystem.
It is desirable that before the next releases this would already be included, starting from the 26th version of the cloud. (AppEcosystem is developed only for 26+ versions)
TODO
Checklist