Skip to content
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.

Sign up for GitHub

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

Jump to bottom

add command to migrate from owncloud wnd external storage #38470

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

add command to migrate from owncloud wnd external storage #38470

wants to merge 3 commits into from

Conversation

icewind1991
Copy link
Member

@icewind1991 icewind1991 commented May 25, 2023 •
edited by artonge
Loading

  • update backend type
  • update storage id
  • reencode password
  • handle non-basic auth providers
    • credentials are stored in oc_external_config, need to re-encrypte the password
      • password::password
    • credentials are stored in oc_credentials, need to migrate them to oc_storages_credentials and re-encrypte the password
      • password::logincredentials
      • password::global
    • Stored in memory so nothing to do
      • password::sessioncredentials
      • kerberos::kerberos
    • Not supported by Nextcloud
      • password::hardcodedconfigcredentials

@icewind1991 icewind1991 mentioned this pull request Jul 27, 2023
Closed
@AndyScherzinger AndyScherzinger added this to the Nextcloud 28 milestone Aug 3, 2023
@AndyScherzinger
Copy link
Member

/backport to stable27

@AndyScherzinger
Copy link
Member

/backport to stable26

@icewind1991 icewind1991 force-pushed the oc-wnd-migrate branch 2 times, most recently from 9516bba to a1dde2f Compare September 5, 2023 13:03
@artonge artonge self-assigned this Oct 2, 2023
@icewind1991 @artonge
add command to migrate from owncloud wnd external storage
7130a04 
Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991 @artonge
re-encode password stored in oc's "v2" format
30c7c47 
Signed-off-by: Robin Appelman <robin@icewind.nl>
@artonge
Add logic to migrate users' credentials
4427fef 
Signed-off-by: Louis Chemineau <louis@chmn.me>
@icewind1991
Copy link
Member Author

@artonge's changes look good 👍

@artonge
Copy link
Contributor

artonge commented Oct 25, 2023

I also pushed more changes here: https://github.com/nextcloud/server/pull/39593/commits

@skjnldsv skjnldsv mentioned this pull request Nov 1, 2023
Merged
This was referenced Nov 6, 2023
Merged
Merged
This was referenced Nov 14, 2023
Merged
Merged
@blizzz blizzz modified the milestones: Nextcloud 28, Nextcloud 29 Nov 23, 2023
@skjnldsv skjnldsv added the 2. developing Work in progress label Feb 21, 2024
This was referenced Mar 12, 2024
Merged
Merged
Merged
@Altahrim Altahrim mentioned this pull request Mar 20, 2024
Merged
@skjnldsv skjnldsv modified the milestones: Nextcloud 29, Nextcloud 30 Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@artonge artonge Awaiting requested review from artonge artonge will be requested when the pull request is marked ready for review artonge is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees

@artonge artonge

Labels
2. developing Work in progress backport-request
Projects
None yet
Milestone
Nextcloud 30
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@icewind1991 @AndyScherzinger @artonge @blizzz @skjnldsv