From 0363ac3a6acc31582f2f70c0801250f976518e44 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Adolfo=20G=C3=B3mez=20Garc=C3=ADa?= <dkmaster@dkmon.com>
Date: Tue, 23 May 2023 03:15:34 +0200
Subject: [PATCH] added /rfx /rfx:gfx to macos by default

---
 server/src/uds/transports/RDP/rdp_file.py | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/server/src/uds/transports/RDP/rdp_file.py b/server/src/uds/transports/RDP/rdp_file.py
index a832da1a7..bbc77967b 100644
--- a/server/src/uds/transports/RDP/rdp_file.py
+++ b/server/src/uds/transports/RDP/rdp_file.py
@@ -48,7 +48,7 @@ class RDPFile:
     address = ''
     username = ''
     domain = ''
-    password = ''
+    password = ''   # nosec: emtpy password is ok here
     redirectSerials = False
     redirectPrinters = False
     redirectDrives = "false"  # Can have "true", "false" or "dynamic"
@@ -179,7 +179,7 @@ class RDPFile:
             params.append('/u:{}'.format(self.username))
         else:
             forceRDPSecurity = True
-        if self.password != '':
+        if self.password:
             params.append('/p:{}'.format(self.password))
         else:
             forceRDPSecurity = True
@@ -192,10 +192,16 @@ class RDPFile:
         if self.customParameters and self.customParameters.strip() != '':
             params += shlex.split(self.customParameters.strip())
 
+        # On MacOSX, /rfx /gfx:rfx are almost inprescindible, as it seems the only way to get a decent performance
+        if self.target == OsDetector.KnownOS.Macintosh:
+            for i in ('/rfx', '/gfx:rfx'):
+                if i not in params:
+                    params.append(i)
+
         return params
 
     def getGeneric(self):  # pylint: disable=too-many-statements
-        password = '{password}'
+        password = '{password}'  # nosec: placeholder
         screenMode = '2' if self.fullScreen else '1'
         audioMode = '0' if self.redirectAudio else '2'
         serials = '1' if self.redirectSerials else '0'