Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
thenextman merged 2 commits into from
Jul 1, 2025
Merged

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

thenextman merged 2 commits into from
Jul 1, 2025

Conversation

@thenextman
Copy link
Member

@thenextman thenextman commented Jul 1, 2025

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.

@thenextman thenextman requested a review from a team July 1, 2025 14:39
@thenextman thenextman enabled auto-merge (squash) July 1, 2025 14:39
@github-actions
Copy link

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

CBenoit
CBenoit reviewed Jul 1, 2025
View reviewed changes
CBenoit
CBenoit approved these changes Jul 1, 2025
View reviewed changes
Copy link
Member

@CBenoit 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 thenextman merged commit 63a0d8c into master Jul 1, 2025
39 checks passed
@thenextman thenextman deleted the pedm-virtual-account-elevator-additional-context branch July 1, 2025 14:58
@CBenoit CBenoit mentioned this pull request Jul 11, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@CBenoit CBenoit CBenoit approved these changes

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@thenextman @CBenoit