Notify fix

README.md

@@ -22,17 +22,20 @@ VM kontextualizálását végzi. A felhő menedzserrel egy virtuális soros vona
    ```
     sudo -i  
     cd /root
-    chmod oug+x .    # fontos a pont!
     mkvirtualenv agent
     workon agent
     git clone https://git.ik.bme.hu/CIRCLE3/agent.git
-    chmod oug+rx agent
     cd agent
     python agent.py --install
    ```
    Ez utóbbi parancs 
      - bemásolja az agent.service-t  /etc/systemd/system -be, engedélyezi, de nem indítja el
      - bemásolja a vm_renewal-t a /usr/bin/ -be 
+ * Célszerűen a cloud usernek van sudo joga jelszó nélkül. 
+ Ha nincs, akkor a /root/.virtualenv/agent és /root/agent katalógusoknak kereshetőknek kell lenniük, és a benne levő 
+ fájloknak olvashatóknak kell lennie a cloud user számára, hogy a vm_reneval működjön. 
+ Ekkor a wm_renewal-bol kivehető a sudo. (Esetleg spec sudo engedéllyel is megoldhato a dolog)  
+ 
 
 ## Windows ##
  * Bundled python alkalmazások, melyekből az első kettő szervízként fut, a harmadik a cloud user belépésekor indul

notify.py

@@ -158,8 +158,9 @@ def open_in_browser(url):
 
     if display:
         import webbrowser
-        webbrowser.open(url, new=2, autoraise=True)
-
+        logger.debug("webbrowser.open(%s)" % url)
+        ret = webbrowser.open(url, new=2, autoraise=True)
+        logger.debug("ret: %s" % ret)
 
 def mount_smb(url):
     data = urlsplit(url)
@@ -201,6 +202,14 @@ def search_display():
     if env:
         return env
 
+    # guess a logged user name
+    user = "cloud"
+    try:
+        process = subprocess.run(["who"], stdout=subprocess.PIPE, stderr=subprocess.DEVNULL, encoding="utf-8", timeout=5)
+        user = process.stdout.split()[0]
+    except Exception:
+        pass
+
     for pid in os.listdir("/proc"):
         if not pid.isdigit():
             continue
@@ -209,14 +218,13 @@ def search_display():
             with open(env, "r") as f:
                 envs = dict(line.split("=", 1)
                             for line in f.read().split("\0") if "=" in line)
-            if "DISPLAY" in envs and ":" in envs["DISPLAY"]:
+            if "DISPLAY" in envs and ":" in envs["DISPLAY"] and "USER" in envs and user == envs["USER"]:
                 p = os.stat(os.path.join("/proc", pid))
                 return envs["DISPLAY"], p.st_uid, p.st_gid
         except Exception:
             continue
     return None
 
-
 if win:
     from twisted.internet import protocol
     from twisted.protocols import basic