Ignores resizes when a desktop has screen_size set

`screen_size` is meant to be a static override of the screen size,
however this property was mistakenly lost when
#39819 was merged.

This commit restores the behavior of `screen_size` by ignoring resizes
when it is set.

Author: Isaiah Becker-Mayer

lib/srv/desktop/rdp/rdpclient/client.go

@@ -239,7 +239,7 @@ func (c *Client) readClientSize() error {
 			continue
 		}
 
-		if c.cfg.Width != 0 && c.cfg.Height != 0 {
+		if c.cfg.hasOverrideSize() {
 			// Some desktops have a screen size in their resource definition.
 			// If non-zero then we always request this screen size.
 			c.cfg.Logger.DebugContext(context.Background(), "Forcing a fixed screen size", "width", c.cfg.Width, "height", c.cfg.Height)
@@ -394,6 +394,12 @@ func (c *Client) startInputStreaming(stopCh chan struct{}) error {
 
 		switch m := msg.(type) {
 		case tdp.ClientScreenSpec:
+			// If the client has specified a fixed screen size, we don't
+			// need to send a screen resize event.
+			if c.cfg.hasOverrideSize() {
+				continue
+			}
+
 			c.cfg.Logger.DebugContext(context.Background(), "Client changed screen size", "width", m.Width, "height", m.Height)
 			if errCode := C.client_write_screen_resize(
 				C.ulong(c.handle),

lib/srv/desktop/rdp/rdpclient/client_common.go

@@ -94,3 +94,10 @@ func (c *Config) checkAndSetDefaults() error {
 	c.Logger = c.Logger.With("rdp-addr", c.Addr)
 	return nil
 }
+
+// hasOverrideSize returns true if the width and height have been set.
+// This will be true when a user has specified a fixed `screen_size` for
+// a given desktop.
+func (c *Config) hasOverrideSize() bool {
+	return c.Width != 0 && c.Height != 0
+}

web/packages/teleport/src/DesktopSession/useTdpClientCanvas.tsx

@@ -126,7 +126,6 @@ export default function useTdpClientCanvas(props: Props) {
   ) => {
     // The first image fragment we see signals a successful TDP connection.
     if (!initialTdpConnectionSucceeded.current) {
-      syncCanvas(ctx.canvas, getDisplaySize());
       setTdpConnection({ status: 'success' });
       initialTdpConnectionSucceeded.current = true;
     }