Skip to content

feat(mstsgu): add extended-auth packet codecs - #1761

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

feat(mstsgu): add extended-auth packet codecs#1761
Marc-André Moreau (mamoreau-devolutions) wants to merge 1 commit into
masterfrom
copilot/gateway-extended-auth-codecs

Conversation

@mamoreau-devolutions

Copy link
Copy Markdown
Contributor

Preserve advertised handshake extended-auth flags, including unknown bits, and encode/decode extended-auth packet blobs with exact lengths.

Add wire vectors for flag combinations, packet round trips, malformed blobs, and u16 blob boundaries.

Preserve advertised handshake extended-auth flags, including unknown bits, and encode/decode extended-auth packet blobs with exact lengths.

Add wire vectors for flag combinations, packet round trips, malformed blobs, and u16 blob boundaries.

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 spec-conformant extended-auth handling to the MSTSGU codec.

Changes:

  • Preserves known and unknown handshake authentication flags.
  • Corrects field widths and adds extended-auth blob codec validation.
  • Adds wire-format, malformed-input, and boundary tests.

No material findings were identified.

Reviewed changes

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

File Description
crates/ironrdp-mstsgu/src/proto.rs Updates extended-auth flags and packet codecs.
crates/ironrdp-mstsgu/tests/http_control.rs Adds protocol vectors and boundary tests.

@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
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