Skip to content

feat(client): enforce gateway redirection policy - #1760

Open
Marc-André Moreau (mamoreau-devolutions) wants to merge 2 commits into
masterfrom
copilot/gateway-policy-enforcement
Open

feat(client): enforce gateway redirection policy#1760
Marc-André Moreau (mamoreau-devolutions) wants to merge 2 commits into
masterfrom
copilot/gateway-policy-enforcement

Conversation

@mamoreau-devolutions

Copy link
Copy Markdown
Contributor

Honor restrictive gateway redirection flags without enabling new channels.

Honor restrictive gateway redirection flags without enabling new channels.
Copilot AI balanced review requested due to automatic review settings August 22, 2026 00:23
@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/S Size: up to 199 counted lines and 5 files; exceeds XS in either measure labels Aug 22, 2026

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

Enforces RD Gateway redirection restrictions for clipboard and drive channels.

Changes:

  • Filters CLIPRDR and RDPDR factories using gateway policy flags.
  • Adds policy interpretation and filtering tests.
  • One issue remains: drive restrictions also disable smart-card redirection.

Comment thread crates/ironrdp-client/src/rdp.rs Outdated
Keep smartcard-only RDPDR when a gateway disables drive redirection.\n\nDiscard initial drive devices and the drive capability instead of suppressing\nthe shared RDPDR channel.
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/S Size: up to 199 counted lines and 5 files; exceeds XS in either measure

Development

Successfully merging this pull request may close these issues.

2 participants