Skip to content

Add support for vmconnect resizing#3776

Open
gpotter2 wants to merge 1 commit into
neutrinolabs:develfrom
gpotter2:session-info
Open

Add support for vmconnect resizing#3776
gpotter2 wants to merge 1 commit into
neutrinolabs:develfrom
gpotter2:session-info

Conversation

@gpotter2
Copy link
Copy Markdown
Contributor

@gpotter2 gpotter2 commented Mar 29, 2026

Hi ! This PR is a followup to #3514 to try and improve the experience on Hyper-V. It enables auto-resizing of the RDP client when used in vmconnect mode, which currently doesn't work.

I was trying to understand why vmconnect.exe (used in Hyper-V scenarios) would allow auto-resizing on Windows but not on Linux. The answer is that it's actually using the OnLoginComplete event to detect when resize is available (https://learn.microsoft.com/en-us/windows/win32/termserv/imstscaxevents-onlogincomplete), because resize isn't available on Windows on the Winlogon screen. This event is triggered when the client receives the INFOTYPE_LOGON* types, as described by [MS-RDPBCGR].

By implementing this message, I was able to have resize work with vmconnect as shown below, once the login screen is passed (this is the same behavior in a Windows VM):

capture.mp4

As a side-effect, completely unrelated, it seems that MSRDC (Windows App / the "Remote Desktop" app) also uses OnLoginComplete to know if another connection is allowed (it only allows 1 login screen at a time). So I believe that this change will also help in that regard.

Comment thread common/ms-rdpbcgr.h Outdated
@gpotter2 gpotter2 force-pushed the session-info branch 2 times, most recently from d270ee9 to df9b755 Compare April 1, 2026 09:54
INFOTYPE_LOGON or a variant is usually sent after the user has
successfully logged in. This triggers a hook in vmconnect that
enables automatic resizing, and also allows msrdc (the other RDP client)
to accept connections to new servers.
@gpotter2
Copy link
Copy Markdown
Contributor Author

gpotter2 commented Apr 7, 2026

I've rebased on master ! Do not hesitate if this needs anything to go further :)

Copy link
Copy Markdown
Member

@matt335672 matt335672 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@metalefty
Copy link
Copy Markdown
Member

Hold on until we finish what we're working on recently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants