Skip to content

node: Don't hold mutex on subscription channels in Spy#4716

Open
djb15 wants to merge 2 commits into
wormhole-foundation:mainfrom
djb15:node/spy-mutex-update
Open

node: Don't hold mutex on subscription channels in Spy#4716
djb15 wants to merge 2 commits into
wormhole-foundation:mainfrom
djb15:node/spy-mutex-update

Conversation

@djb15

@djb15 djb15 commented Mar 12, 2026

Copy link
Copy Markdown
Collaborator

Comment thread node/cmd/spy/spy.go
@pleasew8t pleasew8t force-pushed the node/spy-mutex-update branch from 44e4818 to 591f0c9 Compare March 17, 2026 06:24
Comment thread node/cmd/spy/spy.go

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

Good to approve after the comment is updated

@djb15 djb15 force-pushed the node/spy-mutex-update branch from 45a39f8 to 063a927 Compare May 6, 2026 13:53
johnsaigle
johnsaigle previously approved these changes May 6, 2026
@johnsaigle johnsaigle force-pushed the node/spy-mutex-update branch from 063a927 to 9a5ab9b Compare May 19, 2026 18:38
@djb15 djb15 requested review from bemic and mdulin2 and removed request for fergarrui May 28, 2026 14:33
@djb15 djb15 force-pushed the node/spy-mutex-update branch from 9a5ab9b to 6de2541 Compare June 11, 2026 22:00
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.

3 participants