123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358 |
- diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
- index 9f854b006..6f3546353 100644
- --- a/src/CMakeLists.txt
- +++ b/src/CMakeLists.txt
- @@ -4,7 +4,7 @@ endif()
-
- set(synclib_NAME ${APPLICATION_EXECUTABLE}sync)
-
- -find_package(Qt5 5.6 COMPONENTS Core Network Xml Concurrent WebEngineWidgets WebEngine REQUIRED)
- +find_package(Qt5 5.6 COMPONENTS Core Network Xml Concurrent REQUIRED)
- if (Qt5Core_VERSION VERSION_LESS 5.9.0)
- message(STATUS "For HTTP/2 support, compile with Qt 5.9 or higher.")
- endif()
- diff --git a/src/gui/CMakeLists.txt b/src/gui/CMakeLists.txt
- index 285057992..8e61531bd 100644
- --- a/src/gui/CMakeLists.txt
- +++ b/src/gui/CMakeLists.txt
- @@ -35,14 +35,12 @@ set(client_UI_SRCS
- addcertificatedialog.ui
- proxyauthdialog.ui
- mnemonicdialog.ui
- - wizard/flow2authwidget.ui
- wizard/owncloudadvancedsetuppage.ui
- wizard/owncloudconnectionmethoddialog.ui
- wizard/owncloudhttpcredspage.ui
- wizard/owncloudoauthcredspage.ui
- wizard/owncloudsetupnocredspage.ui
- wizard/owncloudwizardresultpage.ui
- - wizard/webview.ui
- )
-
- set(client_SRCS
- @@ -108,24 +106,17 @@ set(client_SRCS
- creds/credentialsfactory.cpp
- creds/httpcredentialsgui.cpp
- creds/oauth.cpp
- - creds/flow2auth.cpp
- creds/keychainchunk.cpp
- - creds/webflowcredentials.cpp
- - creds/webflowcredentialsdialog.cpp
- wizard/postfixlineedit.cpp
- wizard/abstractcredswizardpage.cpp
- wizard/owncloudadvancedsetuppage.cpp
- wizard/owncloudconnectionmethoddialog.cpp
- wizard/owncloudhttpcredspage.cpp
- wizard/owncloudoauthcredspage.cpp
- - wizard/flow2authcredspage.cpp
- - wizard/flow2authwidget.cpp
- wizard/owncloudsetuppage.cpp
- wizard/owncloudwizardcommon.cpp
- wizard/owncloudwizard.cpp
- wizard/owncloudwizardresultpage.cpp
- - wizard/webviewpage.cpp
- - wizard/webview.cpp
- wizard/slideshow.cpp
- )
-
- @@ -298,7 +289,7 @@ else()
- endif()
-
- add_library(updater STATIC ${updater_SRCS})
- -target_link_libraries(updater ${synclib_NAME} Qt5::Widgets Qt5::Network Qt5::Xml Qt5::WebEngineWidgets)
- +target_link_libraries(updater ${synclib_NAME} Qt5::Widgets Qt5::Network Qt5::Xml)
- target_include_directories(updater PUBLIC ${CMAKE_CURRENT_SOURCE_DIR})
-
- set_target_properties( ${APPLICATION_EXECUTABLE} PROPERTIES
- diff --git a/src/gui/accountmanager.cpp b/src/gui/accountmanager.cpp
- index eec375180..c8ae72bda 100644
- --- a/src/gui/accountmanager.cpp
- +++ b/src/gui/accountmanager.cpp
- @@ -253,20 +253,6 @@ AccountPtr AccountManager::loadAccountHelper(QSettings &settings)
- acc->setUrl(urlConfig.toUrl());
- }
-
- - // Migrate to webflow
- - if (authType == QLatin1String("http")) {
- - authType = "webflow";
- - settings.setValue(QLatin1String(authTypeC), authType);
- -
- - foreach(QString key, settings.childKeys()) {
- - if (!key.startsWith("http_"))
- - continue;
- - auto newkey = QString::fromLatin1("webflow_").append(key.mid(5));
- - settings.setValue(newkey, settings.value((key)));
- - settings.remove(key);
- - }
- - }
- -
- qCInfo(lcAccountManager) << "Account for" << acc->url() << "using auth type" << authType;
-
- acc->_serverVersion = settings.value(QLatin1String(serverVersionC)).toString();
- diff --git a/src/gui/creds/credentialsfactory.cpp b/src/gui/creds/credentialsfactory.cpp
- index 6062f70eb..52e003b57 100644
- --- a/src/gui/creds/credentialsfactory.cpp
- +++ b/src/gui/creds/credentialsfactory.cpp
- @@ -21,7 +21,6 @@
- #ifndef NO_SHIBBOLETH
- #include "creds/shibbolethcredentials.h"
- #endif
- -#include "creds/webflowcredentials.h"
-
- namespace OCC {
-
- @@ -40,8 +39,6 @@ namespace CredentialsFactory {
- } else if (type == "shibboleth") {
- return new ShibbolethCredentials;
- #endif
- - } else if (type == "webflow") {
- - return new WebFlowCredentials;
- } else {
- qCWarning(lcGuiCredentials, "Unknown credentials type: %s", qPrintable(type));
- return new DummyCredentials;
- diff --git a/src/gui/owncloudsetupwizard.cpp b/src/gui/owncloudsetupwizard.cpp
- index 875dacf55..65dea6619 100644
- --- a/src/gui/owncloudsetupwizard.cpp
- +++ b/src/gui/owncloudsetupwizard.cpp
- @@ -409,7 +409,7 @@ void OwncloudSetupWizard::slotAuthError()
-
- // bring wizard to top
- _ocWizard->bringToTop();
- - if (_ocWizard->currentId() == WizardCommon::Page_ShibbolethCreds || _ocWizard->currentId() == WizardCommon::Page_OAuthCreds || _ocWizard->currentId() == WizardCommon::Page_Flow2AuthCreds) {
- + if (_ocWizard->currentId() == WizardCommon::Page_ShibbolethCreds || _ocWizard->currentId() == WizardCommon::Page_OAuthCreds) {
- _ocWizard->back();
- }
- _ocWizard->displayError(errorMsg, _ocWizard->currentId() == WizardCommon::Page_ServerSetup && checkDowngradeAdvised(reply));
- diff --git a/src/gui/wizard/owncloudsetuppage.cpp b/src/gui/wizard/owncloudsetuppage.cpp
- index 9a6ced74b..101b5dae5 100644
- --- a/src/gui/wizard/owncloudsetuppage.cpp
- +++ b/src/gui/wizard/owncloudsetuppage.cpp
- @@ -144,7 +144,7 @@ void OwncloudSetupPage::slotLogin()
- void OwncloudSetupPage::slotGotoProviderList()
- {
- _ocWizard->setRegistration(true);
- - _ocWizard->setAuthType(DetermineAuthTypeJob::AuthType::WebViewFlow);
- + _ocWizard->setAuthType(DetermineAuthTypeJob::AuthType::OAuth);
- _authTypeKnown = true;
- _checking = false;
- emit completeChanged();
- @@ -259,12 +259,8 @@ int OwncloudSetupPage::nextId() const
- return WizardCommon::Page_HttpCreds;
- case DetermineAuthTypeJob::OAuth:
- return WizardCommon::Page_OAuthCreds;
- - case DetermineAuthTypeJob::LoginFlowV2:
- - return WizardCommon::Page_Flow2AuthCreds;
- case DetermineAuthTypeJob::Shibboleth:
- return WizardCommon::Page_ShibbolethCreds;
- - case DetermineAuthTypeJob::WebViewFlow:
- - return WizardCommon::Page_WebView;
- }
- return WizardCommon::Page_HttpCreds;
- }
- diff --git a/src/gui/wizard/owncloudwizard.cpp b/src/gui/wizard/owncloudwizard.cpp
- index 2076136ff..5f3480340 100644
- --- a/src/gui/wizard/owncloudwizard.cpp
- +++ b/src/gui/wizard/owncloudwizard.cpp
- @@ -27,8 +27,6 @@
- #endif
- #include "wizard/owncloudadvancedsetuppage.h"
- #include "wizard/owncloudwizardresultpage.h"
- -#include "wizard/webviewpage.h"
- -#include "wizard/flow2authcredspage.h"
-
- #include "QProgressIndicator.h"
-
- @@ -47,14 +45,12 @@ OwncloudWizard::OwncloudWizard(QWidget *parent)
- , _setupPage(new OwncloudSetupPage(this))
- , _httpCredsPage(new OwncloudHttpCredsPage(this))
- , _browserCredsPage(new OwncloudOAuthCredsPage)
- - , _flow2CredsPage(new Flow2AuthCredsPage)
- #ifndef NO_SHIBBOLETH
- , _shibbolethCredsPage(new OwncloudShibbolethCredsPage)
- #endif
- , _advancedSetupPage(new OwncloudAdvancedSetupPage)
- , _resultPage(new OwncloudWizardResultPage)
- , _credentialsPage(nullptr)
- - , _webViewPage(new WebViewPage(this))
- , _setupLog()
- , _registration(false)
- {
- @@ -62,13 +58,11 @@ OwncloudWizard::OwncloudWizard(QWidget *parent)
- setPage(WizardCommon::Page_ServerSetup, _setupPage);
- setPage(WizardCommon::Page_HttpCreds, _httpCredsPage);
- setPage(WizardCommon::Page_OAuthCreds, _browserCredsPage);
- - setPage(WizardCommon::Page_Flow2AuthCreds, _flow2CredsPage);
- #ifndef NO_SHIBBOLETH
- setPage(WizardCommon::Page_ShibbolethCreds, _shibbolethCredsPage);
- #endif
- setPage(WizardCommon::Page_AdvancedSetup, _advancedSetupPage);
- setPage(WizardCommon::Page_Result, _resultPage);
- - setPage(WizardCommon::Page_WebView, _webViewPage);
-
- connect(this, &QDialog::finished, this, &OwncloudWizard::basicSetupFinished);
-
- @@ -80,11 +74,9 @@ OwncloudWizard::OwncloudWizard(QWidget *parent)
- connect(_setupPage, &OwncloudSetupPage::determineAuthType, this, &OwncloudWizard::determineAuthType);
- connect(_httpCredsPage, &OwncloudHttpCredsPage::connectToOCUrl, this, &OwncloudWizard::connectToOCUrl);
- connect(_browserCredsPage, &OwncloudOAuthCredsPage::connectToOCUrl, this, &OwncloudWizard::connectToOCUrl);
- - connect(_flow2CredsPage, &Flow2AuthCredsPage::connectToOCUrl, this, &OwncloudWizard::connectToOCUrl);
- #ifndef NO_SHIBBOLETH
- connect(_shibbolethCredsPage, &OwncloudShibbolethCredsPage::connectToOCUrl, this, &OwncloudWizard::connectToOCUrl);
- #endif
- - connect(_webViewPage, &WebViewPage::connectToOCUrl, this, &OwncloudWizard::connectToOCUrl);
- connect(_advancedSetupPage, &OwncloudAdvancedSetupPage::createLocalAndRemoteFolders,
- this, &OwncloudWizard::createLocalAndRemoteFolders);
- connect(this, &QWizard::customButtonClicked, this, &OwncloudWizard::skipFolderConfiguration);
- @@ -103,15 +95,6 @@ OwncloudWizard::OwncloudWizard(QWidget *parent)
- setButtonText(QWizard::CustomButton1, tr("Skip folders configuration"));
-
-
- - // Connect styleChanged events to our widgets, so they can adapt (Dark-/Light-Mode switching)
- - connect(this, &OwncloudWizard::styleChanged, _setupPage, &OwncloudSetupPage::slotStyleChanged);
- - connect(this, &OwncloudWizard::styleChanged, _advancedSetupPage, &OwncloudAdvancedSetupPage::slotStyleChanged);
- - connect(this, &OwncloudWizard::styleChanged, _flow2CredsPage, &Flow2AuthCredsPage::slotStyleChanged);
- -
- - customizeStyle();
- -
- - // allow Flow2 page to poll on window activation
- - connect(this, &OwncloudWizard::onActivate, _flow2CredsPage, &Flow2AuthCredsPage::slotPollNow);
- }
-
- void OwncloudWizard::setAccount(AccountPtr account)
- @@ -180,20 +163,12 @@ void OwncloudWizard::successfulStep()
- _browserCredsPage->setConnected();
- break;
-
- - case WizardCommon::Page_Flow2AuthCreds:
- - _flow2CredsPage->setConnected();
- - break;
- -
- #ifndef NO_SHIBBOLETH
- case WizardCommon::Page_ShibbolethCreds:
- _shibbolethCredsPage->setConnected();
- break;
- #endif
-
- - case WizardCommon::Page_WebView:
- - _webViewPage->setConnected();
- - break;
- -
- case WizardCommon::Page_AdvancedSetup:
- _advancedSetupPage->directoriesCreated();
- break;
- @@ -217,10 +192,6 @@ void OwncloudWizard::setAuthType(DetermineAuthTypeJob::AuthType type)
- #endif
- if (type == DetermineAuthTypeJob::OAuth) {
- _credentialsPage = _browserCredsPage;
- - } else if (type == DetermineAuthTypeJob::LoginFlowV2) {
- - _credentialsPage = _flow2CredsPage;
- - } else if (type == DetermineAuthTypeJob::WebViewFlow) {
- - _credentialsPage = _webViewPage;
- } else { // try Basic auth even for "Unknown"
- _credentialsPage = _httpCredsPage;
- }
- @@ -245,7 +216,7 @@ void OwncloudWizard::slotCurrentPageChanged(int id)
- }
-
- setOption(QWizard::HaveCustomButton1, id == WizardCommon::Page_AdvancedSetup);
- - if (id == WizardCommon::Page_AdvancedSetup && (_credentialsPage == _browserCredsPage || _credentialsPage == _flow2CredsPage)) {
- + if (id == WizardCommon::Page_AdvancedSetup) {
- // For OAuth, disable the back button in the Page_AdvancedSetup because we don't want
- // to re-open the browser.
- button(QWizard::BackButton)->setEnabled(false);
- diff --git a/src/gui/wizard/owncloudwizard.h b/src/gui/wizard/owncloudwizard.h
- index 3cbf89f71..2eecbf95a 100644
- --- a/src/gui/wizard/owncloudwizard.h
- +++ b/src/gui/wizard/owncloudwizard.h
- @@ -39,8 +39,6 @@ class OwncloudAdvancedSetupPage;
- class OwncloudWizardResultPage;
- class AbstractCredentials;
- class AbstractCredentialsWizardPage;
- -class WebViewPage;
- -class Flow2AuthCredsPage;
-
- /**
- * @brief The OwncloudWizard class
- @@ -114,11 +112,9 @@ private:
- #ifndef NO_SHIBBOLETH
- OwncloudShibbolethCredsPage *_shibbolethCredsPage;
- #endif
- - Flow2AuthCredsPage *_flow2CredsPage;
- OwncloudAdvancedSetupPage *_advancedSetupPage;
- OwncloudWizardResultPage *_resultPage;
- AbstractCredentialsWizardPage *_credentialsPage;
- - WebViewPage *_webViewPage;
-
- QStringList _setupLog;
-
- diff --git a/src/gui/wizard/owncloudwizardcommon.h b/src/gui/wizard/owncloudwizardcommon.h
- index d1f7c08be..c55ed04fd 100644
- --- a/src/gui/wizard/owncloudwizardcommon.h
- +++ b/src/gui/wizard/owncloudwizardcommon.h
- @@ -38,8 +38,6 @@ namespace WizardCommon {
- Page_HttpCreds,
- Page_ShibbolethCreds,
- Page_OAuthCreds,
- - Page_Flow2AuthCreds,
- - Page_WebView,
- Page_AdvancedSetup,
- Page_Result
- };
- diff --git a/src/libsync/networkjobs.cpp b/src/libsync/networkjobs.cpp
- index 76789a8ce..06c94ea5b 100644
- --- a/src/libsync/networkjobs.cpp
- +++ b/src/libsync/networkjobs.cpp
- @@ -921,7 +921,7 @@ void DetermineAuthTypeJob::start()
- });
- connect(oldFlowRequired, &JsonApiJob::jsonReceived, this, [this](const QJsonDocument &json, int statusCode) {
- if (statusCode == 200) {
- - _resultOldFlow = LoginFlowV2;
- + _resultOldFlow = Basic;
-
- auto data = json.object().value("ocs").toObject().value("data").toObject().value("capabilities").toObject();
- auto gs = data.value("globalscale");
- @@ -929,7 +929,7 @@ void DetermineAuthTypeJob::start()
- auto flow = gs.toObject().value("desktoplogin");
- if (flow != QJsonValue::Undefined) {
- if (flow.toInt() == 1) {
- - _resultOldFlow = WebViewFlow;
- + _resultOldFlow = Basic;
- }
- }
- }
- @@ -955,17 +955,17 @@ void DetermineAuthTypeJob::checkAllDone()
-
- // WebViewFlow > OAuth > Shib > Basic
- if (_account->serverVersionInt() >= Account::makeServerVersion(12, 0, 0)) {
- - result = WebViewFlow;
- + result = Basic;
- }
-
- // LoginFlowV2 > WebViewFlow > OAuth > Shib > Basic
- if (_account->serverVersionInt() >= Account::makeServerVersion(16, 0, 0)) {
- - result = LoginFlowV2;
- + result = Basic;
- }
-
- // If we determined that we need the webview flow (GS for example) then we switch to that
- - if (_resultOldFlow == WebViewFlow) {
- - result = WebViewFlow;
- + if (_resultOldFlow == Basic) {
- + result = Basic;
- }
-
- qCInfo(lcDetermineAuthTypeJob) << "Auth type for" << _account->davUrl() << "is" << result;
- diff --git a/src/libsync/networkjobs.h b/src/libsync/networkjobs.h
- index 765204c88..088c517d7 100644
- --- a/src/libsync/networkjobs.h
- +++ b/src/libsync/networkjobs.h
- @@ -412,8 +412,6 @@ public:
- Basic, // also the catch-all fallback for backwards compatibility reasons
- OAuth,
- Shibboleth,
- - WebViewFlow,
- - LoginFlowV2
- };
-
- explicit DetermineAuthTypeJob(AccountPtr account, QObject *parent = nullptr);
|