Skip to content

feat(activex): prompt for gateway credentials - #1763

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

feat(activex): prompt for gateway credentials#1763
Marc-André Moreau (mamoreau-devolutions) wants to merge 1 commit into
masterfrom
copilot/activex-gateway-credential-prompt

Conversation

@mamoreau-devolutions

Copy link
Copy Markdown
Contributor

Prompt for non-persistent credentials for a configured RD Gateway.

Keep them scoped to the connection attempt and leave cancellation disconnected.

Prompt for non-persistent credentials for a configured RD Gateway.

Keep them scoped to the connection attempt and leave cancellation disconnected.
Copilot AI balanced review requested due to automatic review settings August 22, 2026 03:17
@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 scope/ffi Affects native or .NET bindings 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 non-persistent RD Gateway credential prompting to the ActiveX connection flow.

Changes:

  • Adds a gateway-specific Windows credential prompt.
  • Keeps prompted credentials local to the connection attempt.
  • Leaves canceled prompts disconnected and tests transport mapping.

Comment on lines +7630 to +7631
let mut username = credential_prompt_buffer("", CREDUI_MAX_USERNAME_LENGTH);
let mut password = credential_prompt_buffer("", CREDUI_MAX_PASSWORD_LENGTH);
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 scope/ffi Affects native or .NET bindings 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