Andy Kurth created VCL-959:
------------------------------

             Summary: Update RDP file settings
                 Key: VCL-959
                 URL: https://issues.apache.org/jira/browse/VCL-959
             Project: VCL
          Issue Type: Improvement
          Components: web gui (frontend)
            Reporter: Andy Kurth


The settings that get populated in the .rdp file should be updated.  The 
subroutine that generates the .rdp file is _sendRDPfile_ in _utils.php_.

Each setting should be checked against the current Microsoft reference 
documentation to determine if it's used anymore.  For example, _clear password_ 
does not work when connecting from a modern Windows OS.  It may not work under 
any scenarios anymore.  Even if it did, the security implications outweigh the 
convenience of not having to type the password.

The _winposstr_ is particularly annoying for me.  This is hard-coded as:
{code}print "winposstr:s:0,1,382,71,1182,671\r\n";{code}

The last 4 integers represent distance on screen from left, top, right, and 
bottom.  They should be adjusted for different window heights and widths.  With 
the hard-coded values, when you open the .rdp file you always see a partial 
window with scrollbars.  When connecting from Windows 10 (possibly others 
depending on configuration), the maximize button causes the window to dock in 
the upper-left corner.  It can't be moved.  I always have to manually resize 
the window to its full size so that I can move it where I want it on my screen.

There are some other settings that shouldn't be hard-coded such as:
{code}print "disable wallpaper:i:1\r\n";{code}

The user should be able to control this.

Design-wise, I don't like having these settings stored directly in the user 
table.  A separate table should be created such as _rdpPrefs_.  I can see the 
possible benefit of having similar functionality for different connection 
methods such as Spice, *NX.  Each will have its own parameters.  We should come 
up with a way to store the RDP settings that will work for other methods.







--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to