Do not apply override server url as account url if this is a multi-server JSON string
authorClaudio Cambra <claudio.cambra@nextcloud.com>
Thu, 19 Sep 2024 07:06:01 +0000 (15:06 +0800)
committerbackportbot[bot] <backportbot[bot]@users.noreply.github.com>
Tue, 24 Sep 2024 12:15:37 +0000 (12:15 +0000)
Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>
src/gui/accountmanager.cpp
src/gui/owncloudsetupwizard.cpp

index 4f41267cde77cb49cbb5deca112f3bf0d8e5ccf0..57dfd097bb9cf6278c028683ab7889d5d7d93bb5 100644 (file)
@@ -456,7 +456,8 @@ AccountPtr AccountManager::loadAccountHelper(QSettings &settings)
 
     const auto overrideUrl = Theme::instance()->overrideServerUrl();
     const auto forceAuth = Theme::instance()->forceConfigAuthType();
-    if (!forceAuth.isEmpty() && !overrideUrl.isEmpty()) {
+    const auto multipleOverrideServers = Theme::instance()->multipleOverrideServers();
+    if (!forceAuth.isEmpty() && !overrideUrl.isEmpty() && !multipleOverrideServers) {
         // If forceAuth is set, this might also mean the overrideURL has changed.
         // See enterprise issues #1126
         acc->setUrl(overrideUrl);
index c60fd118ff93e30db57b44686e175196eaa8730b..40b88289a340d4138807f7c121439dd172b15cab 100644 (file)
@@ -103,7 +103,9 @@ void OwncloudSetupWizard::startWizard()
 {
     AccountPtr account = AccountManager::createAccount();
     account->setCredentials(CredentialsFactory::create("dummy"));
-    account->setUrl(Theme::instance()->overrideServerUrl());
+    const auto defaultUrl =
+        Theme::instance()->multipleOverrideServers() ? QString{} : Theme::instance()->overrideServerUrl();
+    account->setUrl(defaultUrl);
     _ocWizard->setAccount(account);
     _ocWizard->setOCUrl(account->url().toString());