Skip to content

feat(mstsgu): handle gateway consent messages - #1762

Merged
Marc-André Moreau (mamoreau-devolutions) merged 3 commits into
masterfrom
copilot/gateway-consent-handling
Aug 23, 2026
Merged

feat(mstsgu): handle gateway consent messages#1762
Marc-André Moreau (mamoreau-devolutions) merged 3 commits into
masterfrom
copilot/gateway-consent-handling

Conversation

@mamoreau-devolutions

Copy link
Copy Markdown
Contributor

Decode gateway consent messages as UTF-16LE during tunnel creation.

Accept consent by default, and let callbacks decline it before tunnel authorization and channel setup.

Decode gateway consent messages as UTF-16LE during tunnel creation.

Accept consent by default, and let callbacks decline it before tunnel authorization and channel setup.
Copilot AI balanced review requested due to automatic review settings August 22, 2026 02:12
@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

Adds MS-TSGU gateway consent decoding and callback-based acceptance before tunnel authorization.

Changes:

  • Decodes UTF-16LE consent messages.
  • Adds consent callbacks and decline errors.
  • Adds documentation and focused tests.

Reviewed changes

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

Show a summary per file
File Description
crates/ironrdp-mstsgu/src/lib.rs Implements consent handling and APIs.
crates/ironrdp-mstsgu/src/test_support.rs Exposes consent evaluation for tests.
crates/ironrdp-mstsgu/tests/consent.rs Tests decoding and callback outcomes.
crates/ironrdp-mstsgu/README.md Documents consent behavior.
crates/ironrdp-mstsgu/Cargo.toml Registers the consent test.

Comment thread crates/ironrdp-mstsgu/src/lib.rs
Comment thread crates/ironrdp-mstsgu/src/lib.rs Outdated
Comment thread crates/ironrdp-mstsgu/README.md Outdated
Comment thread crates/ironrdp-mstsgu/src/lib.rs Outdated
Comment thread crates/ironrdp-mstsgu/src/lib.rs
Allow synchronous consent callbacks to borrow application state and verify that declined consent stops tunnel setup before authorization and channel creation.\n\nDocument the behavior in the crate changelog and README.
@github-actions github-actions Bot added size/M Size: up to 449 counted lines and 10 files; exceeds S in either measure and removed size/S Size: up to 199 counted lines and 5 files; exceeds XS in either measure labels Aug 22, 2026
Require consent callbacks to be Send so existing connection futures remain spawnable while callbacks can still borrow application state during tunnel setup.
@mamoreau-devolutions
Marc-André Moreau (mamoreau-devolutions) merged commit d219ddb into master Aug 23, 2026
43 checks passed
@mamoreau-devolutions
Marc-André Moreau (mamoreau-devolutions) deleted the copilot/gateway-consent-handling branch August 23, 2026 00:29
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/M Size: up to 449 counted lines and 10 files; exceeds S in either measure

Development

Successfully merging this pull request may close these issues.

2 participants