Skip to content

fix(capture): Fix Windows capture stop ctx.#2385

Merged
mereta merged 2 commits into
microsoft:mainfrom
mereta:fix/capturectx
May 26, 2026
Merged

fix(capture): Fix Windows capture stop ctx.#2385
mereta merged 2 commits into
microsoft:mainfrom
mereta:fix/capturectx

Conversation

@mereta

@mereta mereta commented May 26, 2026

Copy link
Copy Markdown
Contributor

Description

Fix Windows capture ctx.

Stop Network capture parent ctx is expired by the time stopNetworkCapture is called.

Please provide a brief description of the changes made in this pull request.

Related Issue

#2298

Checklist

  • I have read the contributing documentation.
  • I signed and signed-off the commits (git commit -S -s ...). See this documentation on signing commits.
  • I have correctly attributed the author(s) of the code.
  • I have tested the changes locally.
  • I have followed the project's style guidelines.
  • I have updated the documentation, if necessary.
  • I have added tests, if applicable.

Screenshots (if applicable) or Testing Completed

image image

Additional Notes

Issue introduced: #2322

https://github.com/microsoft/retina/pull/2322/changes#diff-bd1c3510276019b2f8f85384d4963e4fa366a428acad36280432a841079a9bf2

Signed-off-by: mereta <mereta.degutyte@hotmail.co.uk>
@mereta mereta requested a review from a team as a code owner May 26, 2026 11:49
@mereta mereta enabled auto-merge May 26, 2026 12:20
carlotaarvela
carlotaarvela previously approved these changes May 26, 2026
Signed-off-by: mereta <mereta.degutyte@hotmail.co.uk>
@mereta mereta requested a review from carlotaarvela May 26, 2026 13:53
@mereta mereta added this pull request to the merge queue May 26, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 26, 2026
@mereta mereta added this pull request to the merge queue May 26, 2026
Merged via the queue into microsoft:main with commit d2165ac May 26, 2026
31 checks passed
@mereta mereta deleted the fix/capturectx branch May 26, 2026 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants