Skip to content

fix(daemon): enable gateway client transport - #1827

Open
Marc-André Moreau (mamoreau-devolutions) wants to merge 1 commit into
masterfrom
copilot/daemon-gateway-transport
Open

fix(daemon): enable gateway client transport#1827
Marc-André Moreau (mamoreau-devolutions) wants to merge 1 commit into
masterfrom
copilot/daemon-gateway-transport

Conversation

@mamoreau-devolutions

Copy link
Copy Markdown
Contributor

Enable the client gateway feature in the daemon dependency so gateway PropertySet values reach the daemon client without downstream feature unification.

The daemon now rejects incomplete gateway credentials before starting a session. RDCleanPath and all other transports are unchanged.

Enable the client gateway feature in the daemon dependency so gateway PropertySet values reach the daemon client without downstream feature unification.

The daemon now rejects incomplete gateway credentials before starting a session. RDCleanPath and all other transports are unchanged.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Enables RD Gateway transport for daemon sessions without relying on downstream feature unification.

Changes:

  • Enables ironrdp-client’s gateway feature.
  • Adds gateway credential validation coverage.
  • Updates tests to use the client output-channel abstraction.

Documentation/prose review was not applicable.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
crates/ironrdp-daemon/Cargo.toml Enables gateway client transport.
crates/ironrdp-daemon/src/daemon.rs Tests gateway validation and output events.

@github-actions github-actions Bot added maintainer-required Maintainer review or intervention is required risk/unknown Risk could not be determined automatically; needs maintainer-level scrutiny size/XS Size: up to 49 counted lines and 2 files labels Aug 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer-required Maintainer review or intervention is required risk/unknown Risk could not be determined automatically; needs maintainer-level scrutiny size/XS Size: up to 49 counted lines and 2 files

Development

Successfully merging this pull request may close these issues.

2 participants