Skip to content

fix(pedm): add additional context to virtual account code paths#1409

Merged
Richard Markiewicz (thenextman) merged 2 commits intomasterfrom
pedm-virtual-account-elevator-additional-context
Jul 1, 2025
Merged

fix(pedm): add additional context to virtual account code paths#1409
Richard Markiewicz (thenextman) merged 2 commits intomasterfrom
pedm-virtual-account-elevator-additional-context

Conversation

@thenextman
Copy link
Copy Markdown
Member

Some users are experiencing an error with the virtual account elevator that I cannot reproduce, and we do not have the necessary contextual information. This PR adds additional context to the virtual account elevator code paths.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 1, 2025

Let maintainers know that an action is required on their side

  • Add the label release-required Please cut a new release (Devolutions Gateway, Devolutions Agent, Jetsocat, PowerShell module) when you request a maintainer to cut a new release (Devolutions Gateway, Devolutions Agent, Jetsocat, PowerShell module)

  • Add the label release-blocker Follow-up is required before cutting a new release if a follow-up is required before cutting a new release

  • Add the label publish-required Please publish libraries (`Devolutions.Gateway.Utils`, OpenAPI clients, etc) when you request a maintainer to publish libraries (Devolutions.Gateway.Utils, OpenAPI clients, etc.)

  • Add the label publish-blocker Follow-up is required before publishing libraries if a follow-up is required before publishing libraries

Comment thread crates/devolutions-pedm/src/elevator/virtual_account_elevator.rs Outdated
Copy link
Copy Markdown
Member

@CBenoit Benoît Cortier (CBenoit) left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you!

@thenextman Richard Markiewicz (thenextman) merged commit 63a0d8c into master Jul 1, 2025
39 checks passed
@thenextman Richard Markiewicz (thenextman) deleted the pedm-virtual-account-elevator-additional-context branch July 1, 2025 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants