Merge branch 'dev' into feature/ovpn3-desktop-cli

Improve logic of install_docker.sh

.github/workflows/deploy.yml

@@ -1,7 +1,12 @@
 name: 'Deploy workflow'
 
+on:
+  push:
+    branches:
+      - '**'
 
-on: [push]
+env:
+  QT_MIRROR: https://mirrors.ocf.berkeley.edu/qt/ # https://download.qt.io/static/mirrorlist/
 
 jobs:
   Build-Linux-Ubuntu:
@@ -25,7 +30,7 @@ jobs:
         setup-python: 'true'
         tools: 'tools_ifw'
         set-env: 'true'
-        extra: '--external 7z'
+        extra: '--external 7z --base ${{ env.QT_MIRROR }}'
 
     - name: 'Get sources'
       uses: actions/checkout@v3
@@ -89,7 +94,7 @@ jobs:
         setup-python: 'true'
         tools: 'tools_ifw'
         set-env: 'true'
-        extra: '--external 7z'
+        extra: '--external 7z --base ${{ env.QT_MIRROR }}'
 
     - name: 'Setup mvsc'
       uses: ilammy/msvc-dev-cmd@v1
@@ -119,15 +124,14 @@ jobs:
 
 # ------------------------------------------------------
 
-  Build-IOS:
-    name: 'Build-IOS'
+  Build-iOS:
+    name: 'Build-iOS'
     runs-on: macos-12
 
     env:
       QT_VERSION: 6.5.2
 
     steps:
-    # Just select XCode
     - name: 'Setup xcode'
       uses: maxim-lobanov/setup-xcode@v1
       with:
@@ -143,6 +147,7 @@ jobs:
         arch: 'clang_64'
         dir: ${{ runner.temp }}
         set-env: 'true'
+        extra: '--base ${{ env.QT_MIRROR }}'
 
     - name: 'Install iOS Qt'
       uses: jurplel/install-qt-action@v3
@@ -154,7 +159,7 @@ jobs:
         dir: ${{ runner.temp }}
         setup-python: 'true'
         set-env: 'true'
-        extra: '--external 7z'
+        extra: '--external 7z --base ${{ env.QT_MIRROR }}'
 
     - name: 'Install go'
       uses: actions/setup-go@v3
@@ -174,7 +179,7 @@ jobs:
     - name: 'Setup ccache'
       uses: hendrikmuhs/ccache-action@v1.2
 
-    - name: Install dependencies
+    - name: 'Install dependencies'
       run: pip install jsonschema jinja2
 
     - name: 'Build project'
@@ -232,7 +237,7 @@ jobs:
         setup-python: 'true'
         tools: 'tools_ifw'
         set-env: 'true'
-        extra: '--external 7z'
+        extra: '--external 7z --base ${{ env.QT_MIRROR }}'
 
     - name: 'Get sources'
       uses: actions/checkout@v3
@@ -296,7 +301,7 @@ jobs:
         dir: ${{ runner.temp }}
         setup-python: 'true'
         set-env: 'true'
-        extra: '--external 7z'
+        extra: '--external 7z --base ${{ env.QT_MIRROR }}'
 
     - name: 'Install android Qt'
       uses: jurplel/install-qt-action@v3
@@ -309,7 +314,7 @@ jobs:
         dir: ${{ runner.temp }}
         setup-python: 'true'
         set-env: 'true'
-        extra: '--external 7z'
+        extra: '--external 7z --base ${{ env.QT_MIRROR }}'
 
     - name: 'Grant execute permission for qt-cmake'
       shell: bash

client/core/defs.h

@@ -36,7 +36,7 @@ enum ErrorCode
     ServerPacketManagerError,
 
     // Ssh connection errors
-    SshRequsetDeniedError, SshInterruptedError, SshInternalError,
+    SshRequestDeniedError, SshInterruptedError, SshInternalError,
     SshPrivateKeyError, SshPrivateKeyFormatError, SshTimeoutError,
 
     // Ssh sftp errors
@@ -47,7 +47,6 @@ enum ErrorCode
     SshSftpNoMediaError,
 
     // Local errors
-    FailedToSaveConfigData,
     OpenVpnConfigMissing,
     OpenVpnManagementServerError,
     ConfigMissing,
@@ -67,7 +66,6 @@ enum ErrorCode
 
     // 3rd party utils errors
     OpenSslFailed,
-    OpenVpnExecutableCrashed,
     ShadowSocksExecutableCrashed,
     CloakExecutableCrashed,
 

client/core/errorstrings.cpp

@@ -19,7 +19,7 @@ QString errorString(ErrorCode code){
     case(ServerUserNotInSudo): return QObject::tr("The user does not have permission to use sudo");
 
     // Libssh errors
-    case(SshRequsetDeniedError): return QObject::tr("Ssh request was denied");
+    case(SshRequestDeniedError): return QObject::tr("Ssh request was denied");
     case(SshInterruptedError): return QObject::tr("Ssh request was interrupted");
     case(SshInternalError): return QObject::tr("Ssh internal error");
     case(SshPrivateKeyError): return QObject::tr("Invalid private key or invalid passphrase entered");
@@ -42,7 +42,6 @@ QString errorString(ErrorCode code){
     case(SshSftpNoMediaError): return QObject::tr("Sftp error: No media was in remote drive");
 
     // Local errors
-    case (FailedToSaveConfigData): return QObject::tr("Failed to save config to disk");
     case (OpenVpnConfigMissing): return QObject::tr("OpenVPN config missing");
     case (OpenVpnManagementServerError): return QObject::tr("OpenVPN management server error");
 

client/core/sshclient.cpp

@@ -333,7 +333,7 @@ namespace libssh {
 
         switch (errorCode) {
         case(SSH_NO_ERROR): return ErrorCode::NoError;
-        case(SSH_REQUEST_DENIED): return ErrorCode::SshRequsetDeniedError;
+        case(SSH_REQUEST_DENIED): return ErrorCode::SshRequestDeniedError;
         case(SSH_EINTR): return ErrorCode::SshInterruptedError;
         case(SSH_FATAL): return ErrorCode::SshInternalError;
         default: return ErrorCode::SshInternalError;

client/protocols/openvpnovercloakprotocol.cpp

@@ -22,6 +22,8 @@ OpenVpnOverCloakProtocol::~OpenVpnOverCloakProtocol()
 
 ErrorCode OpenVpnOverCloakProtocol::start()
 {
+
+#if 0
     if (!QFileInfo::exists(cloakExecPath())) {
         setLastError(ErrorCode::CloakExecutableMissing);
         return lastError();
@@ -77,10 +79,12 @@ ErrorCode OpenVpnOverCloakProtocol::start()
 
     if (m_ckProcess.state() == QProcess::ProcessState::Running) {
         setConnectionState(Vpn::ConnectionState::Connecting);
-
+#endif
         return OpenVpnProtocol::start();
+#if 0
     }
     else return ErrorCode::CloakExecutableMissing;
+#endif
 }
 
 void OpenVpnOverCloakProtocol::stop()
@@ -90,16 +94,6 @@ void OpenVpnOverCloakProtocol::stop()
 
     qDebug() << "OpenVpnOverCloakProtocol::stop()";
 
-#ifdef Q_OS_WIN
-    Utils::signalCtrl(m_ckProcess.processId(), CTRL_C_EVENT);
-#endif
-
-    m_ckProcess.terminate();
-
-    if (Utils::processIsRunning(Utils::executable("ck-client", false))) {
-        QThread::msleep(1000);
-        Utils::killProcessByName(Utils::executable("ck-client", false));
-    }
 }
 
 QString OpenVpnOverCloakProtocol::cloakExecPath()

client/protocols/openvpnprotocol.cpp

@@ -10,11 +10,10 @@
 #include "utilities.h"
 #include "version.h"
 
+
 OpenVpnProtocol::OpenVpnProtocol(const QJsonObject &configuration, QObject *parent) : VpnProtocol(configuration, parent)
 {
     readOpenVpnConfiguration(configuration);
-    connect(&m_managementServer, &ManagementServer::readyRead, this,
-            &OpenVpnProtocol::onReadyReadDataFromManagementServer);
 }
 
 OpenVpnProtocol::~OpenVpnProtocol()
@@ -25,7 +24,6 @@ OpenVpnProtocol::~OpenVpnProtocol()
 
 QString OpenVpnProtocol::defaultConfigFileName()
 {
-    // qDebug() << "OpenVpnProtocol::defaultConfigFileName" << defaultConfigPath() + QString("/%1.ovpn").arg(APPLICATION_NAME);
     return defaultConfigPath() + QString("/%1.ovpn").arg(APPLICATION_NAME);
 }
 
@@ -33,25 +31,20 @@ QString OpenVpnProtocol::defaultConfigPath()
 {
     QString p = QStandardPaths::writableLocation(QStandardPaths::AppDataLocation) + "/config";
     Utils::initializePath(p);
-
     return p;
 }
 
 void OpenVpnProtocol::stop()
 {
     qDebug() << "OpenVpnProtocol::stop()";
-    setConnectionState(Vpn::ConnectionState::Disconnecting);
 
     // TODO: need refactoring
     // sendTermSignal() will even return true while server connected ???
     if ((m_connectionState == Vpn::ConnectionState::Preparing) || (m_connectionState == Vpn::ConnectionState::Connecting)
         || (m_connectionState == Vpn::ConnectionState::Connected)
         || (m_connectionState == Vpn::ConnectionState::Reconnecting)) {
-        if (!sendTermSignal()) {
             killOpenVpnProcess();
-        }
-        QThread::msleep(10);
-        m_managementServer.stop();
+            QThread::msleep(10);
     }
     setConnectionState(Vpn::ConnectionState::Disconnected);
 }
@@ -86,9 +79,30 @@ void OpenVpnProtocol::readOpenVpnConfiguration(const QJsonObject &configuration)
 {
     if (configuration.contains(ProtocolProps::key_proto_config_data(Proto::OpenVpn))) {
         QJsonObject jConfig = configuration.value(ProtocolProps::key_proto_config_data(Proto::OpenVpn)).toObject();
-
+        QString plainConfig = jConfig.value(config_key::config).toString().toUtf8();
+        if (configuration.contains(ProtocolProps::key_proto_config_data(Proto::Cloak))) {
+            QJsonObject cloakConfig = configuration.value(ProtocolProps::key_proto_config_data(Proto::Cloak)).toObject();
+            cloakConfig["NumConn"] = 1;
+            cloakConfig["ProxyMethod"] = "openvpn";
+            if (cloakConfig.contains("port")) {
+                int portValue = cloakConfig.value("port").toInt();
+                cloakConfig.remove("port");
+                cloakConfig["RemotePort"] = portValue;
+            }
+            if (cloakConfig.contains("remote")) {
+                QString hostValue = cloakConfig.value("remote").toString();
+                cloakConfig.remove("remote");
+                cloakConfig["RemoteHost"] = hostValue;
+            }
+            plainConfig += "\n<cloak>\n";
+            QJsonDocument Doc(cloakConfig);
+            QByteArray ba = Doc.toJson();
+            QString plainCloak = ba;
+            plainConfig += QString::fromLatin1(plainCloak.toUtf8().toBase64().data());
+            plainConfig += "\n</cloak>\n";
+        }
         m_configFile.open();
-        m_configFile.write(jConfig.value(config_key::config).toString().toUtf8());
+        m_configFile.write(plainConfig.toUtf8());
         m_configFile.close();
         m_configFileName = m_configFile.fileName();
 
@@ -98,7 +112,7 @@ void OpenVpnProtocol::readOpenVpnConfiguration(const QJsonObject &configuration)
 
 bool OpenVpnProtocol::openVpnProcessIsRunning() const
 {
-    return Utils::processIsRunning("openvpn");
+    return Utils::processIsRunning("ovpncli");
 }
 
 void OpenVpnProtocol::disconnectFromManagementServer()
@@ -122,7 +136,6 @@ void OpenVpnProtocol::sendManagementCommand(const QString &command)
 
 uint OpenVpnProtocol::selectMgmtPort()
 {
-
     for (int i = 0; i < 100; ++i) {
         quint32 port = QRandomGenerator::global()->generate();
         port = (double)(65000 - 15001) * port / UINT32_MAX + 15001;
@@ -132,26 +145,63 @@ uint OpenVpnProtocol::selectMgmtPort()
         if (ok)
             return port;
     }
-
     return m_managementPort;
 }
 
 void OpenVpnProtocol::updateRouteGateway(QString line)
 {
-    // TODO: fix for macos
-    line = line.split("ROUTE_GATEWAY", Qt::SkipEmptyParts).at(1);
-    if (!line.contains("/"))
-        return;
-    m_routeGateway = line.split("/", Qt::SkipEmptyParts).first();
-    m_routeGateway.replace(" ", "");
-    qDebug() << "Set VPN route gateway" << m_routeGateway;
+    const QString substr = "sitnl_route_best_gw result: via ";
+    int start = line.indexOf(substr) + substr.size();
+    int end = line.indexOf(" dev ", start);
+
+    m_routeGateway =  line.mid(start, (end-start));
 }
 
+void OpenVpnProtocol::handle_cli_message(QString message)
+{
+    QString line = message;
+
+    if (line.isEmpty()) {
+        return;
+    }
+
+    if (line.contains("EVENT: CONNECTED")) {
+        setConnectionState(Vpn::ConnectionState::Connected);
+    } else if (line.contains("EXITING")) {
+        // openVpnStateSigTermHandler();
+        setConnectionState(Vpn::ConnectionState::Disconnecting);
+    } else if (line.contains("RECONNECTING")) {
+        setConnectionState(Vpn::ConnectionState::Reconnecting);
+    }
+
+    if (line.contains("sitnl_route_best_gw")) {
+        updateRouteGateway(line);
+    }
+
+    if (line.contains("[ifconfig]")) {
+        updateVpnGateway(line);
+    }
+
+    // TODO: SET CORRECT STRING
+    if (line.contains("FATAL")) {
+        if (line.contains("tap-windows6 adapters on this system are currently in use or disabled")) {
+            emit protocolError(ErrorCode::OpenVpnAdaptersInUseError);
+        } else {
+            emit protocolError(ErrorCode::OpenVpnUnknownError);
+        }
+        return;
+    }
+
+}
+
+
 ErrorCode OpenVpnProtocol::start()
 {
     // qDebug() << "Start OpenVPN connection";
     OpenVpnProtocol::stop();
 
+    qDebug() << " Utils::openVpnExecPath();" << Utils::openVpnExecPath();
+
     if (!QFileInfo::exists(Utils::openVpnExecPath())) {
         setLastError(ErrorCode::OpenVpnExecutableMissing);
         return lastError();
@@ -184,23 +234,11 @@ ErrorCode OpenVpnProtocol::start()
     }
 #endif
 
-    //    QString vpnLogFileNamePath = Utils::systemLogPath() + "/openvpn.log";
-    //    Utils::createEmptyFile(vpnLogFileNamePath);
-
-    uint mgmtPort = selectMgmtPort();
-    qDebug() << "OpenVpnProtocol::start mgmt port selected:" << mgmtPort;
-
-    if (!m_managementServer.start(m_managementHost, mgmtPort)) {
-        setLastError(ErrorCode::OpenVpnManagementServerError);
-        return lastError();
-    }
-
     setConnectionState(Vpn::ConnectionState::Connecting);
-
     m_openVpnProcess = IpcClient::CreatePrivilegedProcess();
 
     if (!m_openVpnProcess) {
-        // qWarning() << "IpcProcess replica is not created!";
+        qWarning() << "IpcProcess replica is not created!";
         setLastError(ErrorCode::AmneziaServiceConnectionFailed);
         return ErrorCode::AmneziaServiceConnectionFailed;
     }
@@ -212,122 +250,100 @@ ErrorCode OpenVpnProtocol::start()
         return ErrorCode::AmneziaServiceConnectionFailed;
     }
     m_openVpnProcess->setProgram(PermittedProcess::OpenVPN);
-    QStringList arguments({
-            "--config", configPath(), "--management", m_managementHost, QString::number(mgmtPort),
-            "--management-client" /*, "--log", vpnLogFileNamePath */
+    QStringList arguments({ configPath()/*, "--management", m_managementHost, QString::number(mgmtPort),
+            "--management-client" *//*, "--log", vpnLogFileNamePath */
     });
     m_openVpnProcess->setArguments(arguments);
 
     qDebug() << arguments.join(" ");
     connect(m_openVpnProcess.data(), &PrivilegedProcess::errorOccurred,
-            [&](QProcess::ProcessError error) { qDebug() << "PrivilegedProcess errorOccurred" << error; });
+            [&](QProcess::ProcessError error) {
+                qDebug() << "PrivilegedProcess errorOccurred" << error;
+                setConnectionState(Vpn::ConnectionState::Disconnected);
+    });
 
-    connect(m_openVpnProcess.data(), &PrivilegedProcess::stateChanged,
-            [&](QProcess::ProcessState newState) { qDebug() << "PrivilegedProcess stateChanged" << newState; });
+    connect(m_openVpnProcess.data(), &PrivilegedProcess::stateChanged, [&](QProcess::ProcessState newState) {
+        switch ( newState )
+        {
+            case QProcess::Starting:
+                setConnectionState(Vpn::ConnectionState::Connecting);
+            break;
+            case QProcess::Running:
+                setConnectionState(Vpn::ConnectionState::Connecting);
+            break;
+            default:
+                setConnectionState(Vpn::ConnectionState::Disconnected);
+        }
+        qDebug() << "PrivilegedProcess stateChanged" << newState;
+    });
 
     connect(m_openVpnProcess.data(), &PrivilegedProcess::finished, this,
             [&]() { setConnectionState(Vpn::ConnectionState::Disconnected); });
 
-    m_openVpnProcess->start();
 
-    // startTimeoutTimer();
+    connect(m_openVpnProcess.data(), &PrivilegedProcess::readyRead, this, [&] {
+
+        QRemoteObjectPendingReply<QByteArray> call = m_openVpnProcess->readAll();
+        auto *watcher = new QRemoteObjectPendingCallWatcher(call, this);
+
+        auto *timeoutTimer = new QTimer(this);
+        timeoutTimer->setSingleShot(true);
+        m_watchers.insert(watcher, timeoutTimer);
+
+        connect(timeoutTimer, &QTimer::timeout, this, [this, watcher, timeoutTimer]() {
+            qDebug() << "Foo request timed out.";
+
+            m_watchers.remove(watcher);
+            watcher->deleteLater();
+            timeoutTimer->deleteLater();
+        });
+
+        connect(watcher, &QRemoteObjectPendingCallWatcher::finished, [this](QRemoteObjectPendingCallWatcher *self) {
+            QTimer *timer = m_watchers.take(self);
+            if (timer) {
+                timer->stop();
+                timer->deleteLater();
+            }
+
+            QByteArray result = self->returnValue().toByteArray();
+            handle_cli_message(QString(result));
+            self->deleteLater();
+        });
+
+        timeoutTimer->start(30000);
+
+    });
+
+    connect(m_openVpnProcess.data(), QOverload<int, QProcess::ExitStatus>::of(&PrivilegedProcess::finished), this, [this](int exitCode, QProcess::ExitStatus exitStatus) {
+        qDebug().noquote() << "OpenVPN finished, exitCode, exiStatus" << exitCode << exitStatus;
+        setConnectionState(Vpn::ConnectionState::Disconnected);
+        if (exitStatus != QProcess::NormalExit) {
+            emit protocolError(amnezia::ErrorCode::ShadowSocksExecutableCrashed);
+            stop();
+        }
+        if (exitCode !=0 ) {
+            emit protocolError(amnezia::ErrorCode::InternalError);
+            stop();
+        }
+    });
+
+    m_openVpnProcess->start();
 
     return ErrorCode::NoError;
 }
 
-bool OpenVpnProtocol::sendTermSignal()
-{
-    return m_managementServer.writeCommand("signal SIGTERM");
-}
-
-void OpenVpnProtocol::sendByteCount()
-{
-    m_managementServer.writeCommand("bytecount 1");
-}
-
-void OpenVpnProtocol::sendInitialData()
-{
-    m_managementServer.writeCommand("state on");
-    m_managementServer.writeCommand("log on");
-}
-
-void OpenVpnProtocol::onReadyReadDataFromManagementServer()
-{
-    for (;;) {
-        QString line = m_managementServer.readLine().simplified();
-
-        if (line.isEmpty()) {
-            return;
-        }
-
-        if (!line.contains(">BYTECOUNT")) {
-            qDebug().noquote() << line;
-        }
-
-        if (line.contains(">INFO:OpenVPN Management Interface")) {
-            sendInitialData();
-        } else if (line.startsWith(">STATE")) {
-            if (line.contains("CONNECTED,SUCCESS")) {
-                sendByteCount();
-                stopTimeoutTimer();
-                setConnectionState(Vpn::ConnectionState::Connected);
-                continue;
-            } else if (line.contains("EXITING,SIGTER")) {
-                // openVpnStateSigTermHandler();
-                setConnectionState(Vpn::ConnectionState::Disconnecting);
-                continue;
-            } else if (line.contains("RECONNECTING")) {
-                setConnectionState(Vpn::ConnectionState::Reconnecting);
-                continue;
-            }
-        }
-
-        if (line.contains("ROUTE_GATEWAY")) {
-            updateRouteGateway(line);
-        }
-
-        if (line.contains("PUSH: Received control message")) {
-            updateVpnGateway(line);
-        }
-
-        if (line.contains("FATAL")) {
-            if (line.contains("tap-windows6 adapters on this system are currently in use or disabled")) {
-                emit protocolError(ErrorCode::OpenVpnAdaptersInUseError);
-            } else {
-                emit protocolError(ErrorCode::OpenVpnUnknownError);
-            }
-            return;
-        }
-
-        QByteArray data(line.toStdString().c_str());
-        if (data.contains(">BYTECOUNT:")) {
-            int beg = data.lastIndexOf(">BYTECOUNT:");
-            int end = data.indexOf("\n", beg);
-
-            beg += sizeof(">BYTECOUNT:") - 1;
-            QList<QByteArray> count = data.mid(beg, end - beg + 1).split(',');
-
-            quint64 r = static_cast<quint64>(count.at(0).trimmed().toULongLong());
-            quint64 s = static_cast<quint64>(count.at(1).trimmed().toULongLong());
-
-            setBytesChanged(r, s);
-        }
-    }
-}
-
 void OpenVpnProtocol::updateVpnGateway(const QString &line)
 {
-    // line looks like
-    // PUSH: Received control message: 'PUSH_REPLY,route 10.8.0.1,topology net30,ping 10,ping-restart
-    // 120,ifconfig 10.8.0.6 10.8.0.5,peer-id 0,cipher AES-256-GCM'
-
-    QStringList params = line.split(",");
-    for (const QString &l : params) {
-        if (l.contains("ifconfig")) {
+    // "[ifconfig] [10.8.0.14] [10.8.0.13]"
+    QStringList params = line.split("\n");
+    for (const QString &param : params) {
+        if (param.contains("ifconfig")) {
+            QString l = param.right(param.size() - param.indexOf("ifconfig"));
             if (l.split(" ").size() == 3) {
                 m_vpnLocalAddress = l.split(" ").at(1);
+                m_vpnLocalAddress.remove("[");m_vpnLocalAddress.remove("]");
                 m_vpnGateway = l.split(" ").at(2);
-
+                m_vpnGateway.remove("[");m_vpnGateway.remove("]");
                 qDebug() << QString("Set vpn local address %1, gw %2").arg(m_vpnLocalAddress).arg(vpnGateway());
             }
         }

client/protocols/openvpnprotocol.h

@@ -25,8 +25,6 @@ public:
     static QString defaultConfigFileName();
     static QString defaultConfigPath();
 
-protected slots:
-    void onReadyReadDataFromManagementServer();
 
 private:
     QString configPath() const;
@@ -34,6 +32,7 @@ private:
     bool sendTermSignal();
     void readOpenVpnConfiguration(const QJsonObject &configuration);
     void disconnectFromManagementServer();
+    void handle_cli_message(QString message);
     void killOpenVpnProcess();
     void sendByteCount();
     void sendInitialData();
@@ -42,6 +41,7 @@ private:
     const QString m_managementHost = "127.0.0.1";
     const unsigned int m_managementPort = 57775;
 
+    QHash<QRemoteObjectPendingCallWatcher*, QTimer*> m_watchers;
     ManagementServer m_managementServer;
     QString m_configFileName;
     QTemporaryFile m_configFile;

client/server_scripts/install_docker.sh

@@ -1,19 +1,19 @@
-if which apt-get > /dev/null 2>&1; then pm=$(which apt-get); docker_pkg="docker.io"; dist="debian";\
-elif which dnf > /dev/null 2>&1; then pm=$(which dnf); docker_pkg="docker"; dist="fedora";\
-elif which yum > /dev/null 2>&1; then pm=$(which yum); docker_pkg="docker"; dist="centos";\
+if which apt-get > /dev/null 2>&1; then pm=$(which apt-get); silent_inst="-yq install"; check_pkgs="-yq update"; docker_pkg="docker.io"; dist="debian";\
+elif which dnf > /dev/null 2>&1; then pm=$(which dnf); silent_inst="-yq install"; check_pkgs="-yq check-update"; docker_pkg="docker"; dist="fedora";\
+elif which yum > /dev/null 2>&1; then pm=$(which yum); silent_inst="-y -q install"; check_pkgs="-y -q check-update"; docker_pkg="docker"; dist="centos";\
 else echo "Packet manager not found"; exit 1; fi;\
-echo "Dist: $dist, Packet manager: $pm, Docker pkg: $docker_pkg";\
+echo "Dist: $dist, Packet manager: $pm, Install command: $silent_inst, Check pkgs command: $check_pkgs, Docker pkg: $docker_pkg";\
 if [ "$dist" = "debian" ]; then export DEBIAN_FRONTEND=noninteractive; fi;\
-if ! command -v sudo > /dev/null 2>&1; then $pm update -yq; $pm install -yq sudo; fi;\
-if ! command -v fuser > /dev/null 2>&1; then sudo $pm install -yq psmisc; fi;\
-if ! command -v lsof > /dev/null 2>&1; then sudo $pm install -yq lsof; fi;\
-if ! command -v docker > /dev/null 2>&1; then sudo $pm update -yq; sudo $pm install -yq $docker_pkg;\
+if ! command -v sudo > /dev/null 2>&1; then $pm $check_pkgs; $pm $silent_inst sudo; fi;\
+if ! command -v fuser > /dev/null 2>&1; then sudo $pm $check_pkgs; sudo $pm $silent_inst psmisc; fi;\
+if ! command -v lsof > /dev/null 2>&1; then sudo $pm $check_pkgs; sudo $pm $silent_inst lsof; fi;\
+if ! command -v docker > /dev/null 2>&1; then sudo $pm $check_pkgs; sudo $pm $silent_inst $docker_pkg;\
   if [ "$dist" = "fedora" ] || [ "$dist" = "centos" ] || [ "$dist" = "debian" ]; then sudo systemctl enable docker && sudo systemctl start docker; fi;\
 fi;\
 if [ "$dist" = "debian" ]; then \
   docker_service=$(systemctl list-units --full --all | grep docker.service | grep -v inactive | grep -v dead | grep -v failed);\
-  if [ -z "$docker_service" ]; then sudo $pm update -yq; sudo $pm install -yq curl $docker_pkg; fi;\
+  if [ -z "$docker_service" ]; then sudo $pm $check_pkgs; sudo $pm $silent_inst curl $docker_pkg; fi;\
   sleep 3 && sudo systemctl start docker && sleep 3;\
 fi;\
-if ! command -v sudo > /dev/null 2>&1; then echo "Failed to install Docker";exit 1;fi;\
+if ! command -v sudo > /dev/null 2>&1; then echo "Failed to install Docker"; exit 1; fi;\
 docker --version

client/utilities.cpp

@@ -226,13 +226,13 @@ QStringList Utils::summarizeRoutes(const QStringList &ips, const QString cidr)
 QString Utils::openVpnExecPath()
 {
 #ifdef Q_OS_WIN
-    return Utils::executable("openvpn/openvpn", true);
+    return Utils::executable("openvpn/ovpncli", true);
 #elif defined Q_OS_LINUX
     // We have service that runs OpenVPN on Linux. We need to make same
     // path for client and service.
-    return Utils::executable("../../client/bin/openvpn", true);
+    return Utils::executable("../../client/bin/ovpncli", true);
 #else
-    return Utils::executable("/openvpn", true);
+    return Utils::executable("/ovpncli", true);
 #endif
 }
 

service/server/tapcontroller_win.cpp

@@ -261,7 +261,7 @@ QString TapController::getTapInstallPath()
 
 QString TapController::getOpenVpnPath()
 {
-    return qApp->applicationDirPath() + "\\openvpn\\openvpn.exe";
+    return qApp->applicationDirPath() + "\\openvpn\\ovpncli.exe";
 }
 
 QString TapController::getTapDriverDir()