ATBUpdateTool.ini

@@ -1,5 +1,5 @@
 [REPOSITORY_URL]
-repository-url="gitea@ptu-config.atb-comm.de:ATB/"
+repository-url="https://git.mimbach49.de/GerhardHoffmann"
 
 [DIRECTORIES]
 plugin-directory="/usr/lib/"

DownloadDCFirmware/main.cpp

@@ -55,7 +55,7 @@ int main(int argc, char **argv) {
     openlog("ATB-UPDATE-DC-FIRMWARE", LOG_PERROR | LOG_PID | LOG_CONS, LOG_USER);
 
     QApplication a(argc, argv);
-    QApplication::setApplicationName("ATBDownloadDCFirmware");
+    QApplication::setApplicationName("ATBUpdateTool");
     QApplication::setApplicationVersion(APP_VERSION);
 
     if (!messageHandlerInstalled()) { // change internal qt-QDebug-handling
@@ -124,12 +124,11 @@ int main(int argc, char **argv) {
     MainWindow mw;
 
     mw.setWindowFlags(Qt::Window | Qt::FramelessWindowHint);
-    // mw.showFullScreen();
+    //mw.showFullScreen();
 
-    // qCritical() << "SHOW";
+    qCritical() << "SHOW";
 
-    // mw.show();
+    mw.show();
 
-    return 0;
+    return a.exec();
-    // return a.exec();
 }

DownloadDCJsonFiles/update.cpp

@@ -414,6 +414,10 @@ bool Update::downloadJson(enum FileTypeJson type,
         }
     }
 
+    m_hw->dc_autoRequest(false);
+    qDebug() << "SET AUTO-REQUEST=FALSE";
+    QThread::sleep(1);  // make sure the auto-request flag is acknowledged
+
     return ret;
 }
 

UpdatePTUDevCtrl/UpdatePTUDevCtrl.pro

@@ -132,16 +132,7 @@ DEFINES += QT_DEPRECATED_WARNINGS
 #           update.
 #           In case there is an automatic update (00:00-04:00) use M0100 instead
 #           of U0002.
-# 1.4.10  : Fix: for the time being, never set the autoRequest-flag to false.
+VERSION="1.4.9"
-# 1.4.11  : Fix: remove hard-coded path to git-repos of gitea in mimbach. Read
-#           from ATBUpdateTool.ini the url for the gitrepositories.
-# 1.5.0   : Use ptu-config.atb-comm.de (given in ini-file) as new server for
-#           customer repositories.
-#           Fix: allow empty lines in opkg_commands.
-# 1.5.1   : Fix: do not use cleanPath() on a url-address.
-# 1.5.2   : Remove .ipk and .gz files in /var/cache/opkg.
-# 1.5.3   : Build customer_(id) name without right justification.
-VERSION="1.5.3"
 # PLANNED TODOS:
 #   1:  Das Repository wird repariert bwz. neu geklont. Unabhaengig vom WAIT.
 #   2:  Wenn der WAIT-Button aktiv ist, dann wird ein Repository repariert (neu

UpdatePTUDevCtrl/git/git_client.cpp

@@ -9,15 +9,14 @@
 #include <QStringList>
 
 
-GitClient::GitClient(QString const &customerRepositoryPath,
+GitClient::GitClient(QString const &customerNrStr,
-                     QString const &customerNrStr,
                      QString const &customerRepository,
                      QString const &workingDirectory,
                      QString const &branchName,
                      QObject *parent)
   : QObject(parent)
   , m_worker(qobject_cast<Worker *>(parent))
-  , m_repositoryPath(customerRepositoryPath)
+  , m_repositoryPath(QString("https://git.mimbach49.de/GerhardHoffmann/%1.git").arg(customerNrStr))
   , m_customerNr(customerNrStr)
   , m_workingDirectory(workingDirectory)
   , m_branchName(branchName)
@@ -393,45 +392,6 @@ std::optional<QString> GitClient::gitPull() {
     if (QDir(m_customerRepository).exists()) {
         qInfo() << "BRANCH NAME" << m_branchName;
 
-#if 0
-        IMPORTANT COMMENT:
-
-        If remote host keys are changed, then
-
-        export GIT_SSH_COMMAND="ssh -i /opt/app/tools/atbupdate/.keys/id_ed25519_ptuConfig"
-        git pull
-
-        leads to the following warning/error message:
-
-        @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
-        @    WARNING: REMOTE HOST IDENTIFICATION HAS CHANGED!     @
-        @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
-        IT IS POSSIBLE THAT SOMEONE IS DOING SOMETHING NASTY!
-        Someone could be eavesdropping on you right now (man-in-the-middle attack)!
-        It is also possible that a host key has just been changed.
-        The fingerprint for the ECDSA key sent by the remote host is
-        SHA256:vOD5jF2hglGktqLhK9ABxfEjwEgIK68/v9erdT05NDQ.
-        Please contact your system administrator.
-        Add correct host key in /home/root/.ssh/known_hosts to get rid of this message.
-        Offending ECDSA key in /home/root/.ssh/known_hosts:1
-        Password authentication is disabled to avoid man-in-the-middle attacks.
-        Keyboard-interactive authentication is disabled to avoid man-in-the-middle attacks.
-        Agent forwarding is disabled to avoid man-in-the-middle attacks.
-        X11 forwarding is disabled to avoid man-in-the-middle attacks.
-        Already up to date.
-
-        This first part is from ssh itself. Only the last line is the git message.
-
-        Here an output of running ATBUpdateTool with a corrupted known-hosts-file:
-
-        ...
-        Oct 23 14:18:18 ATB_PTU5 ATBUpdateTool[2696]: ( branchExistsRemotely : 310 ) branch "zg1/zone1" EXISTS REMOTELY. ( "@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@\r\n@    WARNING: REMOTE HOST IDENTIFICATION HAS CHANGED!     @\r\n@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@\r\nIT IS POSSIBLE THAT SOMEONE IS DOING SOMETHING NASTY!\r\nSomeone could be eavesdropping on you right now (man-in-the-middle attack)!\r\nIt is also possible that a host key has just been changed.\r\nThe fingerprint for the ECDSA key sent by the remote host is\nSHA256:vOD5jF2hglGktqLhK9ABxfEjwEgIK68/v9erdT05NDQ.\r\nPlease contact your system administrator.\r\nAdd correct host key in /home/root/.ssh/known_hosts to get rid of this message.\r\nOffending ECDSA key in /home/root/.ssh/known_hosts:1\r\nPassword authentication is disabled to avoid man-in-the-middle attacks.\r\nKeyboard-interactive authentication is disabled to avoid man-in-the-middle attacks.\r\nAgent forwarding is disabled to avoid man-in-the-middle attacks.\r\nX11 forwarding is disabled to avoid man-in-the-middle attacks.\r\nbd4e8da4780b1a7d6be3d3ce8419f43ccf7e706f\trefs/heads/zg1/zone1" )
-        Oct 23 14:18:18 ATB_PTU5 ATBUpdateTool[2696]: EXECUTED "git branch -l" "(runtime 16ms)" with code 0 IN "/opt/app/tools/atbupdate/customer_336"
-        Oct 23 14:18:18 ATB_PTU5 ATBUpdateTool[2696]: "UPDATE_STEP::PULL_NEW_BRANCH"
-        Oct 23 14:18:18 ATB_PTU5 ATBUpdateTool[2696]: "BRANCH-NAME zg1/zone1 CONTAINED IN RESULT master\n* zg1/zone1" .....
-
-        The download continues.
-#endif
         Command c("git pull");
         if (c.execute(m_customerRepository)) {
             QString const s = c.getCommandResult().trimmed();

UpdatePTUDevCtrl/git/git_client.h

@@ -22,8 +22,7 @@ class GitClient : public QObject {
     bool copyGitConfigFromMaster();
 
   public:
-    explicit GitClient(QString const &customerRepositoryUrl,
+    explicit GitClient(QString const &customerNrStr,
-                       QString const &customerNrStr,
                        QString const &repositoryPath,
                        QString const &workingDirectory = QCoreApplication::applicationDirPath(),
                        QString const &branchName = "master",

UpdatePTUDevCtrl/main.cpp

@@ -50,11 +50,9 @@
 // argv[1]: file to send to dc
 int main(int argc, char *argv[]) {
     QByteArray const value = qgetenv("LC_ALL");
-    if (value.isEmpty() || value != "C") {
+    if (value != "C") {
         qputenv("LC_ALL", "C");
     }
-
-
     // qputenv("XDG_RUNTIME_DIR", "/var/run/user/0");
 
     openlog("ATB-UPDATE", LOG_PERROR | LOG_PID | LOG_CONS, LOG_USER);
@@ -74,38 +72,6 @@ int main(int argc, char *argv[]) {
     parser.readSettings();
 
     QString repositoryUrl = parser.repositoryUrl();
-
-    if (repositoryUrl.endsWith('/')) {
-        repositoryUrl.chop(1);
-    }
-
-    QString gitSSHCommand("");
-
-    if (repositoryUrl.contains("ptu-config.atb-comm.de")) {
-        QByteArray const v = qgetenv("GIT_SSH_COMMAND");
-        if (v.isEmpty()) {
-            QString sshKeyFile("/opt/app/tools/atbupdate/.keys/id_ed25519_ptuConfig");
-            if (QFileInfo(sshKeyFile).exists()) {
-                gitSSHCommand = "ssh -i /opt/app/tools/atbupdate/.keys/id_ed25519_ptuConfig";
-                if (!qputenv("GIT_SSH_COMMAND", QByteArray(gitSSHCommand.toStdString().c_str()))) {
-                    qCritical() << "ERROR: GIT_SSH_COMMAND not put into env. Exiting...";
-                    return -1;
-                }
-            } else {
-                qCritical() << "ERROR ssh-key-file" << sshKeyFile << "does not exists. Exiting...";
-                return -1;
-            }
-        } else {
-            gitSSHCommand = QString(v.toStdString().c_str());
-            qCritical() << "WARNING GIT_SSH_COMMAND already set in enviroment:"
-                        << gitSSHCommand;
-            if (gitSSHCommand != "ssh -i /opt/app/tools/atbupdate/.keys/id_ed25519_ptuConfig") {
-                qCritical() << "ERROR" << gitSSHCommand << "wrong. Exiting...";
-                return -1;
-            }
-        }
-    }
-
     QString plugInDir = parser.plugInDir();
     QString plugInName = parser.plugInName();
     QString workingDir = parser.workingDir();
@@ -128,9 +94,6 @@ int main(int argc, char *argv[]) {
 
     qInfo() << "pwd ......................" << rtPath;
     qInfo() << "repositoryUrl ............" << repositoryUrl;
-    if (!gitSSHCommand.isEmpty()) {
-        qInfo() << "GIT_SSH_COMMAND .........." << gitSSHCommand;
-    }
     qInfo() << "plugInDir ................" << plugInDir;
     qInfo() << "plugInName ..............." << plugInName;
     qInfo() << "workingDir ..............." << workingDir;

UpdatePTUDevCtrl/update.cpp

@@ -193,8 +193,8 @@ bool Update::openSerial(int br, QString baudrate, QString comPort) const {
                       + " " + baudrate + " " + comPort + "...OK");
 
             // m_hw->dc_autoRequest(true);
-            // m_hw->dc_autoRequest(false);
+            m_hw->dc_autoRequest(false);
-            // QThread::sleep(1);
+            QThread::sleep(1);
 
             Utils::printInfoMsg(QString("IS PORT OPEN %1").arg(m_hw->dc_isPortOpen()));
             return true;
@@ -507,9 +507,9 @@ bool Update::downloadJson(enum FileTypeJson type,
             }
         }
 
-        // m_hw->dc_autoRequest(false);
+        m_hw->dc_autoRequest(false);
-        // qDebug() << "SET AUTO-REQUEST=FALSE";
+        qDebug() << "SET AUTO-REQUEST=FALSE";
-        // QThread::sleep(1);  // make sure the auto-request flag is acknowledged
+        QThread::sleep(1);  // make sure the auto-request flag is acknowledged
     }
 
     return true;
@@ -584,8 +584,8 @@ QStringList Update::getDcSoftAndHardWareVersion() {
         QString const &hwVersion = m_hw->dc_getHWversion().toLower().trimmed();
         QString const &swVersion = m_hw->dc_getSWversion().toLower().trimmed();
 
-        //m_hw->dc_autoRequest(false);
+        m_hw->dc_autoRequest(false);
-        //QThread::sleep(1); // make sure the timer-slots are inactive
+        QThread::sleep(1); // make sure the timer-slots are inactive
 
         if (!hwVersion.isEmpty() && !swVersion.isEmpty()) {
             return QStringList() << hwVersion << swVersion;

UpdatePTUDevCtrl/utils.cpp

@@ -110,19 +110,19 @@ QString Utils::getTariffInfo(QString fileName) {
 }
 
 QString Utils::zoneName(quint8 i) {
-    //static constexpr char const *zName[] = {
+    static constexpr char const *zName[] = {
-    //    "",
+        "",
-    //    "purple",
+        "purple",
-    //    "blue",
+        "blue",
-    //    "yellow",
+        "yellow",
-    //    "green",
+        "green",
-    //    "yellow (mars)",
+        "yellow (mars)",
-    //    "green (mars)"
+        "green (mars)"
-    //};
+    };
-    //if (i < (sizeof(zName)/sizeof(char const *))) {
+    if (i < (sizeof(zName)/sizeof(char const *))) {
-    //    return zName[i];
+        return zName[i];
-    //}
+    }
-    return "---";
+    return "N/A";
 }
 
 void Utils::printCriticalErrorMsg(QString const &errorMsg, bool upper, bool lower) {

UpdatePTUDevCtrl/worker.cpp

@@ -141,7 +141,7 @@ Worker::Worker(int customerNr,
                char const *serialInterface,
                char const *baudrate)
   : m_customerNr(customerNr)
-  , m_customerNrStr(QString("customer_") + QString::number(m_customerNr))
+  , m_customerNrStr(QString("customer_") + QString::number(m_customerNr).rightJustified(3, '0'))
   , m_machineNr(machineNr)
   , m_zoneNr(zoneNr)
   , m_pluginDir(pluginDir)
@@ -157,7 +157,7 @@ Worker::Worker(int customerNr,
   , m_parent(parent)
   , m_serialInterface(serialInterface)
   , m_baudrate(baudrate)
-  , m_gc(m_customerRepositoryPath, m_customerNrStr, m_customerRepository, m_workingDirectory, m_branchName, this)
+  , m_gc(m_customerNrStr, m_customerRepository, m_workingDirectory, m_branchName, this)
   , m_versionInfo(QStringList())
   , m_osVersion(getOsVersion())
   , m_atbqtVersion(getATBQTVersion())
@@ -456,14 +456,6 @@ void Worker::privateUpdate() {
     //
     ////////////////////////////////////////////////////////////////////////////
     if ((continueUpdate = customerEnvironment()) == false) {
-        // even if something goes wrong creating the environment, try to execute
-        // opkg_commands
-        if (QDir(m_customerRepository).exists()) {
-            // always execute contents of opkg_commands-file
-            m_filesToUpdate.clear();
-            m_filesToUpdate << "etc/psa_update/opkg_commands";
-            execOpkgCommands();
-        }
         return;
     }
     m_versionInfo = m_gc.gitShowReason(m_branchName);
@@ -478,14 +470,6 @@ void Worker::privateUpdate() {
     //
     ////////////////////////////////////////////////////////////////////////////
     if ((continueUpdate = filesToUpdate()) == false) {
-        // even if something goes wrong in filesToUpdate, try to execute
-        // opkg_commands
-        if (QDir(m_customerRepository).exists()) {
-            // always execute contents of opkg_commands-file
-            m_filesToUpdate.clear();
-            m_filesToUpdate << "etc/psa_update/opkg_commands";
-            execOpkgCommands();
-        }
         return;
     }
     m_versionInfo = m_gc.gitShowReason(m_branchName);
@@ -499,14 +483,6 @@ void Worker::privateUpdate() {
     //
     ////////////////////////////////////////////////////////////////////////////
     if ((continueUpdate = syncCustomerRepositoryAndFS()) == false) {
-        // even if something goes wrong with rsync, try to execute
-        // opkg_commands
-        if (QDir(m_customerRepository).exists()) {
-            // always execute contents of opkg_commands-file
-            m_filesToUpdate.clear();
-            m_filesToUpdate << "etc/psa_update/opkg_commands";
-            execOpkgCommands();
-        }
         return;
     }
     lst = QStringList(QString(smap[UPDATE_STEP::SYNC_CUSTOMER_REPOSITORY_SUCCESS]));
@@ -888,24 +864,7 @@ bool Worker::computeFilesToDownload() {
     return (m_filesToDownload.size() > 0);
 }
 
-
-bool Worker::cleanUpOpkgCache() {
-    bool removedFiles = true;
-    QDir dir("/var/cache/opkg");
-    if (dir.exists()) {
-        dir.setNameFilters(QStringList() << ".gz" << ".ipk");
-        dir.setFilter(QDir::Files);
-        foreach(QString dirFile, dir.entryList()) {
-            removedFiles &= dir.remove(dirFile);
-        }
-    }
-    return removedFiles;
-}
-
 bool Worker::execOpkgCommands() {
-    if (!cleanUpOpkgCache()) {
-        CONSOLE() << "INFO: some cached opkg files not removed";
-    }
     for (int i = 0; i < m_filesToUpdate.size(); ++i) {
         QString const fName = m_filesToUpdate.at(i);
         if (fName.contains("opkg_commands", Qt::CaseInsensitive)) {
@@ -921,12 +880,10 @@ bool Worker::execOpkgCommands() {
                         while (!in.atEnd()) {
                             QString line = in.readLine();
                             // TODO: "^\\s*[#]{0,}$" : empty line or comment line starting with #
-                            static const QRegularExpression comment("^\\s*[#].*$");
+                            static const QRegularExpression comment("^\\s*#.*$");
-                            static const QRegularExpression emptyLine("^\\s*$");
+                            if (line.indexOf(comment, 0) == -1) {
-                            if (line.indexOf(emptyLine, 0) == -1 &&
+                                // found opkg command
-                                line.indexOf(comment, 0) == -1) {
                                 QString opkgCommand = line.trimmed();
-                                qCritical() << "Found opkg-command" << opkgCommand;
                                 if (!executeOpkgCommand(opkgCommand)) {
                                     opkgErrorLst << opkgCommand;
                                 } else {

UpdatePTUDevCtrl/worker.h

@@ -193,7 +193,6 @@ class Worker : public QThread{
     QString m_apismVersion;
 
     bool executeOpkgCommand(QString opkgCommand);
-    bool cleanUpOpkgCache();
     QString getOsVersion() const;
     QString getATBQTVersion() const;
     QString getATBUpdateToolVersion() const;