Packaging changes

CHANGELOG.md

@@ -6,49 +6,6 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [5.9.0] - 2025-06-16
-
-### Added
-
-- Added changes for oggit
-
-## [5.8.1] - 2025-06-13
-
-### Fixed
-
-- Improve fail condition when no network is detected
-
-## [5.8.0] - 2025-06-12
-
-### Changed
-
-- Agents aren't being sent any signals on user logout. On the server side, assume that client disconnection == logout
-
-## [5.7.1] - 2025-06-05
-
-### Fixed
-
-- Correcly handle UNIX signals in the user instance of the agent
-
-## [5.7.0] - 2025-05-27
-
-### Changed
-
-- Use TLS again
-
-## [5.6.0] - 2025-05-21
-
-### Changed
-
-- Launch QT6 browser
-- Change URLs using dbus
-
-## [5.5.0] - 2025-05-19
-
-### Changed
-
-- Revert to the QT4 browser again
-
 ## [5.4.0] - 2025-05-19
 
 ### Changed

linux/build-packages.sh

@@ -3,4 +3,4 @@
 cd $(dirname "$0")
 
 # Debian based
-debuild -D --build=binary --post-clean --lintian-opts --profile debian
+debuild -D -us -uc --build=binary --post-clean --lintian-opts --profile debian

linux/debian/changelog

@@ -1,45 +1,8 @@
-ogagent (5.9.0-1) stable; urgency=medium
+ogagent (5.4.1-1) stable; urgency=medium
 
-  * Add changes for oggit
+  * Oggit support
 
- -- OpenGnsys developers <info@opengnsys.es>  Mon, 16 Jun 2025 11:12:55 +0200
-
-ogagent (5.8.1-1) stable; urgency=medium
-
-  * Improve fail condition when no network is detected
-
- -- OpenGnsys developers <info@opengnsys.es>  Fri, 13 Jun 2025 10:01:43 +0200
-
-ogagent (5.8.0-1) stable; urgency=medium
-
-  * When client disconnect, assume that the user logged out
-
- -- OpenGnsys developers <info@opengnsys.es>  Thu, 12 Jun 2025 15:30:50 +0200
-
-ogagent (5.7.1-1) stable; urgency=medium
-
-  * Correctly handle UNIX signals
-
- -- OpenGnsys developers <info@opengnsys.es>  Thu, 05 Jun 2025 12:07:30 +0200
-
-ogagent (5.7.0-1) stable; urgency=medium
-
-  * Use TLS again
-
- -- OpenGnsys developers <info@opengnsys.es>  Wed, 21 May 2025 17:39:13 +0200
-
-ogagent (5.6.0-1) stable; urgency=medium
-
-  * Execute 'launch_browser' rather than 'browser'
-  * Change URLs using dbus
-
- -- OpenGnsys developers <info@opengnsys.es>  Wed, 21 May 2025 15:06:52 +0200
-
-ogagent (5.5.0-1) stable; urgency=medium
-
-  * Return to the QT4 browser again
-
- -- OpenGnsys developers <info@opengnsys.es>  Mon, 19 May 2025 10:57:37 +0200
+ -- OpenGnsys developers <info@opengnsys.es>  Mon, 29 May 2025 09:46:42 +0200
 
 ogagent (5.4.0-1) stable; urgency=medium
 

src/OGAgentUser.py

@@ -35,8 +35,6 @@ import json
 import sys
 import time
 import os
-import socket
-import signal
 from PyQt6 import QtCore, QtGui, QtWidgets
 
 from about_dialog_ui import Ui_OGAAboutDialog
@@ -330,29 +328,6 @@ if __name__ == '__main__':
         trayIcon.quit()
         sys.exit(1)
 
-    ## begin SIGTERM handling
-    signal_socket = socket.socketpair()
-    signal_socket[0].setblocking(False)
-    signal_socket[1].setblocking(False)
-    signal.set_wakeup_fd(signal_socket[0].fileno())
-
-    def signal_handler(signum, frame):
-        #print (f"Received signal {signum}")
-        pass
-
-    def qt_signal_handler():
-        data = signal_socket[1].recv(1)
-        #print(f"Signal ({data}) received via socket, shutting down gracefully...")
-        if trayIcon:
-            trayIcon.quit()
-
-    signal.signal(signal.SIGTERM, signal_handler)
-    signal.signal(signal.SIGINT, signal_handler)
-
-    notifier = QtCore.QSocketNotifier(signal_socket[1].fileno(), QtCore.QSocketNotifier.Type.Read)
-    notifier.activated.connect(qt_signal_handler)
-    ## end SIGTERM handling
-
     app.aboutToQuit.connect(trayIcon.cleanup)
     trayIcon.show()
 

src/VERSION

@@ -1 +1 @@
-5.9.0
+5.4.0

src/cfg/ogagent.cfg

@@ -20,11 +20,9 @@ log=DEBUG
 imgname=
 
 # TLS
-# The agent will look for these files in /opt/opengnsys/etc, /usr/share/OGAgent,
-# windows "Program Files (x86)" and the current working directory
-ca=ca.crt
-crt=ogagent.crt
-key=ogagent.key
+ca=C:\Program Files (x86)\OGagent\ca.crt
+crt=C:\Program Files (x86)\OGagent\ogagent.crt
+key=C:\Program Files (x86)\OGagent\ogagent.key
 
 
 # Module specific

src/opengnsys/RESTApi.py

@@ -45,7 +45,6 @@ from .log import logger
 from .utils import exceptionToMessage
 
 TIMEOUT = 5  # Connection timout, in seconds
-VERIFY_TLS=True
 
 
 class RESTError(Exception):
@@ -96,7 +95,6 @@ class REST(object):
         @param url The url of the REST API Base. The trailing '/' can be included or omitted, as desired.
         """
         self.endpoint = url
-        global VERIFY_TLS
 
         if self.endpoint[-1] != '/':
             self.endpoint += '/'
@@ -111,47 +109,21 @@ class REST(object):
             logger.debug ('TLS not available: python requests library is old')
 
         self.use_tls = url.startswith ('https')
-        if self.use_tls:
-            if not ca_file or not crt_file or not key_file:
-                raise Exception ('missing TLS parameters in REST constructor')
-
-            certs_dirs = ['/opt/opengnsys/etc', '/usr/share/OGAgent']
-            pf = os.environ.get ('PROGRAMFILES(X86)')
-            if pf: certs_dirs.append (os.path.join (pf, 'OGAgent'))
-            certs_dirs.append (os.getcwd())
-            certs_dir = None
-            for sp in certs_dirs:
-                if os.path.exists (sp):
-                    logger.debug (f'Looking for TLS files in ({sp})')
-                    certs_dir = sp
-                    break
-
-            if not certs_dir:
-                logger.debug ("Don't know where to look for TLS files")
-                errs = 1
-            else:
-                errs = 0
-                for f in [ca_file, crt_file, key_file]:
-                    if os.path.exists (f'{certs_dir}/{f}'):
-                        logger.debug (f'{certs_dir}/{f}: found')
-                    else:
-                        logger.error (f'{f}: No such file or directory')
-                        errs += 1
-
-            if errs:
-                self.verify_tls = False
-                logger.debug ('HTTP client: using insecure TLS to talk to ogcore due to missing files')
-            else:
-                self.ca_file  = f'{certs_dir}/{ca_file}'
-                self.crt_file = f'{certs_dir}/{crt_file}'
-                self.key_file = f'{certs_dir}/{key_file}'
-                self.verify_tls = VERIFY_TLS
-                if self.verify_tls:
-                    logger.debug ('HTTP client: using TLS to talk to ogcore')
-                else:
-                    logger.debug ('HTTP client: using insecure TLS as requested to talk to ogcore')
-        else:
-            logger.debug ('HTTP client: not using TLS to talk to ogcore')
+        #if self.use_tls:
+        #    if not ca_file or not crt_file or not key_file:
+        #        raise Exception ('missing TLS parameters in REST constructor')
+        #
+        #    errs = 0
+        #    for f in [ca_file, crt_file, key_file]:
+        #        if not os.path.exists (f):
+        #            logger.error (f'{f}: No such file or directory')
+        #            errs += 1
+        #    if errs:
+        #        raise Exception ('TLS files not found')
+        #
+        #self.ca_file  = ca_file
+        #self.crt_file = crt_file
+        #self.key_file = key_file
 
         # Disable logging requests messages except for errors, ...
         logging.getLogger("requests").setLevel(logging.CRITICAL)
@@ -184,11 +156,7 @@ class REST(object):
                 # Old requests version does not support verify, but it do not checks ssl certificate by default
                 if self.newerRequestLib:
                     if self.use_tls:
-                        if self.verify_tls:
-                            r = requests.get(url, cert=(self.crt_file, self.key_file), verify=self.ca_file, timeout=TIMEOUT)
-                        else:
-                            logger.warning ('using insecure TLS for GET')
-                            r = requests.get(url, verify=False, timeout=TIMEOUT)
+                        r = requests.get(url, verify=False, timeout=TIMEOUT)
                     else:
                         r = requests.get(url, timeout=TIMEOUT)
                 else:
@@ -197,11 +165,7 @@ class REST(object):
                 logger.debug('Requesting using POST {}, data: {}'.format(url, data))
                 if self.newerRequestLib:
                     if self.use_tls:
-                        if self.verify_tls:
-                            r = requests.post(url, data=data, headers={'content-type': 'application/json'}, cert=(self.crt_file, self.key_file), verify=self.ca_file, timeout=TIMEOUT)
-                        else:
-                            logger.warning ('using insecure TLS for POST')
-                            r = requests.post(url, data=data, headers={'content-type': 'application/json'}, verify=False, timeout=TIMEOUT)
+                        r = requests.post(url, data=data, headers={'content-type': 'application/json'}, verify=False, timeout=TIMEOUT)
                     else:
                         r = requests.post(url, data=data, headers={'content-type': 'application/json'}, timeout=TIMEOUT)
                 else:

src/opengnsys/httpserver.py

@@ -43,7 +43,6 @@ from .utils import exceptionToMessage
 from .certs import createSelfSignedCert
 from .log import logger
 
-VERIFY_TLS=True
 
 class HTTPServerHandler(BaseHTTPRequestHandler):
     service = None
@@ -154,46 +153,15 @@ class HTTPThreadingServer(ThreadingMixIn, HTTPServer):
 class HTTPServerThread(threading.Thread):
     def __init__(self, address, service):
         super(self.__class__, self).__init__()
-        global VERIFY_TLS
 
         HTTPServerHandler.service = service  # Keep tracking of service so we can intercact with it
 
+        self.certFile = createSelfSignedCert()
         self.server = HTTPThreadingServer(address, HTTPServerHandler)
         context = ssl.SSLContext(ssl.PROTOCOL_TLS_SERVER)
-
-        pf = os.environ.get ('PROGRAMFILES(X86)')
-        if pf: pf = os.path.join (pf, 'OGAgent')
-        if os.path.exists ('/opt/opengnsys/etc/ogagent.crt') and os.path.exists ('/opt/opengnsys/etc/ogagent.key') and os.path.exists ('/opt/opengnsys/etc/ca.crt'):
-            logger.debug ('HTTP server: using certificate/CA from /opt/opengnsys/etc')
-            context.load_cert_chain (certfile='/opt/opengnsys/etc/ogagent.crt', keyfile='/opt/opengnsys/etc/ogagent.key')
-            context.load_verify_locations (cafile='/opt/opengnsys/etc/ca.crt')
-        elif pf and os.path.exists (os.path.join (pf, 'ogagent.crt')) and os.path.exists (os.path.join (pf, 'ogagent.key')) and os.path.exists (os.path.join (pf, 'ca.crt')):
-            logger.debug (f'HTTP server: using certificate/CA from the installation path ({pf})')
-            context.load_cert_chain (certfile=os.path.join (pf, 'ogagent.crt'), keyfile=os.path.join (pf, 'ogagent.key'))
-            context.load_verify_locations (cafile=os.path.join (pf, 'ca.crt'))
-        elif os.path.exists ('./ogagent.crt') and os.path.exists ('./ogagent.key') and os.path.exists ('./ca.crt'):
-            cwd = os.getcwd()
-            logger.debug (f'HTTP server: using certificate/CA from the current working directory ({cwd})')
-            context.load_cert_chain (certfile=f'{cwd}/ogagent.crt', keyfile=f'{cwd}/ogagent.key')
-            context.load_verify_locations (cafile=f'{cwd}/ca.crt')
-        else:
-            logger.debug ('HTTP server: using a self-signed certificate')
-            self.certFile = createSelfSignedCert()
-            context.load_cert_chain (certfile=self.certFile)
-            VERIFY_TLS = False
-
-        if VERIFY_TLS:
-            context.verify_mode = ssl.CERT_REQUIRED
-            context.verify_flags &= ssl.VERIFY_X509_STRICT
-        else:
-            context.verify_mode = ssl.CERT_NONE
-            context.verify_flags &= ~ssl.VERIFY_X509_STRICT
-
-        s = context.cert_store_stats()
-        if 'x509_ca' in s: logger.debug (f'HTTP server: {s['x509_ca']} CAs loaded')
-        if 'x509'    in s: logger.debug (f'HTTP server: {s['x509']} certs loaded')
+        context.load_cert_chain(certfile=self.certFile)
         self.server.socket = context.wrap_socket(self.server.socket, server_side=True)
-
+        
         logger.debug('Initialized HTTPS Server thread on {}'.format(address))
 
     def getServerUrl(self):

src/opengnsys/ipc.py

@@ -30,7 +30,6 @@
 """
 
 
-import os
 import json
 import queue
 import socket
@@ -194,8 +193,6 @@ class ClientProcessor(threading.Thread):
                 logger.error('Invalid message in queue: {}'.format(e))
 
         logger.debug('Client processor stopped')
-        if os.path.exists ('/windows/temp'): open ('/windows/temp/ogagentuser_died', 'w').close()
-        else:                                open (         '/tmp/ogagentuser_died', 'w').close()
         try:
             self.clientSocket.close()
         except Exception:

src/opengnsys/modules/server/OpenGnSys/__init__.py

@@ -147,11 +147,7 @@ class OpenGnSysWorker(ServerWorker):
         for t in range(0, 300):
             try:
                 # Get the first network interface
-                nets = list (operations.getNetworkInfo())
-                if 0 == len (nets):
-                    logger.error ('No network interfaces found')
-                    raise Exception ('No network interfaces found')
-                self.interface = nets[0]
+                self.interface = list(operations.getNetworkInfo())[0]
             except Exception as e:
                 # Wait 1 sec. and retry
                 logger.warn (e)

src/opengnsys/service.py

@@ -30,7 +30,6 @@
 @author: Adolfo Gómez, dkmaster at dkmon dot com
 """
 
-import os
 import json
 import socket
 import time
@@ -198,16 +197,6 @@ class CommonService(object):
         Invoked to wait a bit
         CAN be OVERRIDDEN
         """
-        client_died=False
-        if os.path.exists ('/windows/temp/ogagentuser_died'):
-            os.unlink ('/windows/temp/ogagentuser_died')
-            client_died=True
-        elif os.path.exists ('/tmp/ogagentuser_died'):
-            os.unlink ('/tmp/ogagentuser_died')
-            client_died=True
-        if client_died:
-            self.notifyLogout (b'')
-
         time.sleep(float(miliseconds) / 1000)
 
     def notifyStop(self):

src/opengnsys/workers/oglive_worker.py

@@ -33,8 +33,6 @@
 import os
 import re
 import time
-try: import dbus     ## don't fail on windows (the worker will later refuse to load anyway)
-except: pass
 import random
 import subprocess
 import threading
@@ -371,25 +369,17 @@ class ogLiveWorker(ServerWorker):
 
     def cargaPaginaWeb (self, url=None):
         if (not url): url = self.urlMenu
+        os.system ('pkill -f -9 browser')
+        os.system ('pkill -f -9 OGBrowser')
+        os.system ('pkill -f -9 QtWebEngineProcess')
 
-        dbus_address = os.environ.get ('DBUS_SESSION_BUS_ADDRESS')
-        if not dbus_address: logger.warning ('env var DBUS_SESSION_BUS_ADDRESS not set, cargaPaginaWeb() will likely not work')
-
-        b = dbus.SystemBus()
-        dest = 'es.opengnsys.OGBrowser.browser'
-        path = '/'
-        interface = None
-        method = 'setURL'
-        signature = 's'
+        p = subprocess.Popen (['/usr/bin/launch_browser', url])
         try:
-            b.call_blocking (dest, path, interface, method, 's', [url])
-        except Exception as e:
-            if 'ServiceUnknown' in str(e):
-                ## browser not running
-                subprocess.Popen (['/usr/bin/launch_browser', url])
-            else:
-                logger.error (f'Error al cambiar URL: ({e})')
-                return False
+            p.wait (2)       ## if the process dies before 2 seconds...
+            logger.error ('Error al ejecutar OGBrowser, return code "{}"'.format (p.returncode))
+            return False
+        except subprocess.TimeoutExpired:
+            pass
 
         return True