[Pkg-owncloud-commits] [owncloud-client] 132/332: ownCloudcmd: Use Account object and ConnectionValidator.

Sandro Knauß hefee-guest at moszumanska.debian.org
Thu Aug 14 21:06:50 UTC 2014


This is an automated email from the git hooks/post-receive script.

hefee-guest pushed a commit to branch master
in repository owncloud-client.

commit 5f96de32bb602d51d5ea30084f05c62c12637dc2
Author: Klaas Freitag <freitag at owncloud.com>
Date:   Tue Jun 24 15:33:42 2014 +0200

    ownCloudcmd: Use Account object and ConnectionValidator.
    
    That reads the credentials from the mirall config file if it was not
    defined on the command line. Moreover, the connection is validated
    before, which sets up the credentials properly.
---
 src/owncloudcmd/owncloudcmd.cpp | 244 +++++++++++++++++++++++-----------------
 src/owncloudcmd/owncloudcmd.h   |  36 +++++-
 2 files changed, 171 insertions(+), 109 deletions(-)

diff --git a/src/owncloudcmd/owncloudcmd.cpp b/src/owncloudcmd/owncloudcmd.cpp
index 23a110f..ca61557 100644
--- a/src/owncloudcmd/owncloudcmd.cpp
+++ b/src/owncloudcmd/owncloudcmd.cpp
@@ -25,42 +25,152 @@
 #include "mirall/syncengine.h"
 #include "mirall/syncjournaldb.h"
 #include "mirall/logger.h"
-#include "csync.h"
-#include "mirall/clientproxy.h"
-#include "mirall/account.h"
+
 #include "creds/httpcredentials.h"
 #include "owncloudcmd.h"
 #include "simplesslerrorhandler.h"
 
-using namespace Mirall;
 
-struct CmdOptions {
-    QString source_dir;
-    QString target_url;
-    QString config_directory;
-    QString proxy;
-    bool silent;
-    bool trustSSL;
-};
+OwncloudCmd::OwncloudCmd(CmdOptions options)
+ : QObject(), _options(options)
+{
+
+}
 
-int getauth(const char* prompt, char* buf, size_t len, int a, int b, void *userdata)
+void OwncloudCmd::slotConnectionValidatorResult(ConnectionValidator::Status stat)
 {
-    (void) a;
-    (void) b;
+    // call csync_create even if the connect fails, since csync_destroy is
+    // called later in any case, and that needs a proper initialized csync_ctx.
+    if( csync_create( &_csync_ctx, _options.source_dir.toUtf8(),
+                      _options.target_url.toUtf8()) < 0 ) {
+        qCritical("Unable to create csync-context!");
+        emit( finished() );
+        return;
+    }
+
+    if( stat != ConnectionValidator::Connected ) {
+        qCritical("Connection cound not be established!");
+        emit( finished() );
+        return;
+
+    }
+
+    int rc = ne_sock_init();
+    if (rc < 0) {
+        qCritical("ne_sock_init failed!");
+        emit( finished() );
+        return;
+    }
+
+    csync_set_userdata(_csync_ctx, &_options);
+
+    if( csync_init( _csync_ctx ) < 0 ) {
+        qCritical("Could not initialize csync!");
+        _csync_ctx = 0;
+        emit( finished() );
+        return;
+    }
 
-    struct CmdOptions *opts = (struct CmdOptions*) userdata;
+    csync_set_module_property(_csync_ctx, "csync_context", _csync_ctx);
+    if( !_options.proxy.isNull() ) {
+        QString host;
+        int port = 0;
+        bool ok;
+        // Set as default and let overwrite later
+        csync_set_module_property(_csync_ctx, "proxy_type", (void*) "NoProxy");
+
+        QStringList pList = _options.proxy.split(':');
+        if(pList.count() == 3) {
+            // http: //192.168.178.23 : 8080
+            //  0            1            2
+            host = pList.at(1);
+            if( host.startsWith("//") ) host.remove(0, 2);
 
-    std::cout << "** Authentication required: \n" << prompt << std::endl;
-    std::string s;
-    if(opts && opts->trustSSL) {
-        s = "yes";
+            port = pList.at(2).toInt(&ok);
+
+            if( !host.isNull() ) {
+                csync_set_module_property(_csync_ctx, "proxy_type", (void*) "HttpProxy");
+                csync_set_module_property(_csync_ctx, "proxy_host", host.toUtf8().data());
+                if( ok && port ) {
+                    csync_set_module_property(_csync_ctx, "proxy_port", (void*) &port);
+                }
+            }
+        }
     } else {
-        std::getline(std::cin, s);
+        _clientProxy.setupQtProxyFromConfig();
+        QString url( _options.target_url );
+        if( url.startsWith("owncloud")) {
+            url.remove(0, 8);
+            url = QString("http%1").arg(url);
+        }
+        _clientProxy.setCSyncProxy(QUrl(url), _csync_ctx);
     }
-    strncpy( buf, s.c_str(), len );
-    return 0;
+
+    SyncJournalDb *db = new SyncJournalDb(_options.source_dir);
+    SyncEngine *engine = new SyncEngine(_csync_ctx, _options.source_dir, QUrl(_options.target_url).path(), _folder, db);
+    connect( engine, SIGNAL(finished()), this, SIGNAL(finished()) );
+    QObject::connect(engine, SIGNAL(transmissionProgress(Progress::Info)), this, SLOT(transmissionProgressSlot()));
+
+    // Have to be done async, else, an error before exec() does not terminate the event loop.
+    QMetaObject::invokeMethod(engine, "startSync", Qt::QueuedConnection);
+
 }
 
+bool OwncloudCmd::runSync()
+{
+    QUrl url(_options.target_url.toUtf8());
+
+    _account = new Account;
+
+    // Find the folder and the original owncloud url
+    QStringList splitted = url.path().split(_account->davPath());
+    url.setPath(splitted.value(0));
+    url.setScheme(url.scheme().replace("owncloud", "http"));
+    _folder = splitted.value(1);
+
+    SimpleSslErrorHandler *sslErrorHandler = new SimpleSslErrorHandler;
+
+    csync_set_log_level(_options.silent ? 1 : 11);
+    Logger::instance()->setLogFile("-");
+
+    if( url.userInfo().isEmpty() ) {
+        // If there was no credentials coming in commandline url
+        // than restore the credentials from a configured client
+        delete _account;
+        _account = Account::restore();
+    } else {
+        // set the commandline credentials
+        _account->setCredentials(new HttpCredentials(url.userName(), url.password()));
+    }
+
+    if (_account) {
+        _account->setUrl(url);
+        _account->setSslErrorHandler(sslErrorHandler);
+        AccountManager::instance()->setAccount(_account);
+
+        _conValidator = new ConnectionValidator(_account);
+        connect( _conValidator, SIGNAL(connectionResult(ConnectionValidator::Status)),
+                 this, SLOT(slotConnectionValidatorResult(ConnectionValidator::Status)) );
+        _conValidator->checkConnection();
+    } else {
+        // no account found
+        return false;
+    }
+
+    return true;
+}
+
+void OwncloudCmd::destroy()
+{
+    csync_destroy(_csync_ctx);
+}
+
+void OwncloudCmd::transmissionProgressSlot()
+{
+    // do something nice here.
+}
+
+
 void help()
 {
     std::cout << "owncloudcmd - command line ownCloud client tool." << std::endl;
@@ -136,98 +246,20 @@ int main(int argc, char **argv) {
     CmdOptions options;
     options.silent = false;
     options.trustSSL = false;
-    ClientProxy clientProxy;
 
     parseOptions( app.arguments(), &options );
 
 
-    QUrl url(options.target_url.toUtf8());
-    Account account;
-
-    // Find the folder and the original owncloud url
-    QStringList splitted = url.path().split(account.davPath());
-    url.setPath(splitted.value(0));
-    url.setScheme(url.scheme().replace("owncloud", "http"));
-    QString folder = splitted.value(1);
-
-    SimpleSslErrorHandler *sslErrorHandler = new SimpleSslErrorHandler;
-
-    account.setUrl(url);
-    account.setCredentials(new HttpCredentials(url.userName(), url.password()));
-    account.setSslErrorHandler(sslErrorHandler);
-    AccountManager::instance()->setAccount(&account);
-
-    CSYNC *_csync_ctx;
-    if( csync_create( &_csync_ctx, options.source_dir.toUtf8(),
-                      options.target_url.toUtf8()) < 0 ) {
-        qFatal("Unable to create csync-context!");
-        return EXIT_FAILURE;
-    }
-    int rc = ne_sock_init();
-    if (rc < 0) {
-        qFatal("ne_sock_init failed!");
-    }
-
-    csync_set_log_level(options.silent ? 1 : 11);
-    Logger::instance()->setLogFile("-");
-
-    csync_set_userdata(_csync_ctx, &options);
-    csync_set_auth_callback( _csync_ctx, getauth );
+    OwncloudCmd owncloudCmd(options);
 
-    if( csync_init( _csync_ctx ) < 0 ) {
-        qFatal("Could not initialize csync!");
-        return EXIT_FAILURE;
+    QObject::connect(&owncloudCmd, SIGNAL(finished()), &app, SLOT(quit()));
+    if( !owncloudCmd.runSync() ) {
+        return 1;
     }
 
-    csync_set_module_property(_csync_ctx, "csync_context", _csync_ctx);
-    if( !options.proxy.isNull() ) {
-        QString host;
-        int port = 0;
-        bool ok;
-
-        // Set as default and let overwrite later
-        csync_set_module_property(_csync_ctx, "proxy_type", (void*) "NoProxy");
-
-        QStringList pList = options.proxy.split(':');
-        if(pList.count() == 3) {
-            // http: //192.168.178.23 : 8080
-            //  0            1            2
-            host = pList.at(1);
-            if( host.startsWith("//") ) host.remove(0, 2);
-
-            port = pList.at(2).toInt(&ok);
-
-            if( !host.isNull() ) {
-                csync_set_module_property(_csync_ctx, "proxy_type", (void*) "HttpProxy");
-                csync_set_module_property(_csync_ctx, "proxy_host", host.toUtf8().data());
-                if( ok && port ) {
-                    csync_set_module_property(_csync_ctx, "proxy_port", (void*) &port);
-                }
-            }
-        }
-    } else {
-        clientProxy.setupQtProxyFromConfig();
-        QString url( options.target_url );
-        if( url.startsWith("owncloud")) {
-            url.remove(0, 8);
-            url = QString("http%1").arg(url);
-        }
-        clientProxy.setCSyncProxy(QUrl(url), _csync_ctx);
-    }
-
-    OwncloudCmd owncloudCmd;
-
-    SyncJournalDb db(options.source_dir);
-    SyncEngine engine(_csync_ctx, options.source_dir, QUrl(options.target_url).path(), folder, &db);
-    QObject::connect(&engine, SIGNAL(finished()), &app, SLOT(quit()));
-    QObject::connect(&engine, SIGNAL(transmissionProgress(Progress::Info)), &owncloudCmd, SLOT(transmissionProgressSlot()));
-
-    // Have to be done async, else, an error before exec() does not terminate the event loop.
-    QMetaObject::invokeMethod(&engine, "startSync", Qt::QueuedConnection);
-
     app.exec();
 
-    csync_destroy(_csync_ctx);
+    owncloudCmd.destroy();
 
     ne_sock_exit();
 
diff --git a/src/owncloudcmd/owncloudcmd.h b/src/owncloudcmd/owncloudcmd.h
index ae55305..4b6836b 100644
--- a/src/owncloudcmd/owncloudcmd.h
+++ b/src/owncloudcmd/owncloudcmd.h
@@ -18,14 +18,44 @@
 
 #include <QObject>
 
+#include "csync.h"
+
+#include "mirall/connectionvalidator.h"
+#include "mirall/clientproxy.h"
+#include "mirall/account.h"
+
+using namespace Mirall;
+
+struct CmdOptions {
+    QString source_dir;
+    QString target_url;
+    QString config_directory;
+    QString proxy;
+    bool silent;
+    bool trustSSL;
+};
 
 class OwncloudCmd : public QObject {
     Q_OBJECT
 public:
-    OwncloudCmd() : QObject() { }
+    OwncloudCmd(CmdOptions options);
+    bool runSync();
+    void destroy();
+
 public slots:
-    void transmissionProgressSlot() {
-    }
+    void slotConnectionValidatorResult(ConnectionValidator::Status stat);
+    void transmissionProgressSlot();
+
+signals:
+    void finished();
+
+private:
+    CmdOptions _options;
+    ConnectionValidator *_conValidator;
+    CSYNC *_csync_ctx;
+    Account *_account;
+    ClientProxy _clientProxy;
+    QString _folder;
 };
 
 #endif

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-owncloud/owncloud-client.git



More information about the Pkg-owncloud-commits mailing list