Skip to content

Fix a race condition with DataChannel - #421

Open
gtherond wants to merge 4 commits into
gradio-app:mainfrom
gtherond:fix/DataChannel
Open

Fix a race condition with DataChannel#421
gtherond wants to merge 4 commits into
gradio-app:mainfrom
gtherond:fix/DataChannel

Conversation

@gtherond

@gtherond gtherond commented Jan 7, 2026

Copy link
Copy Markdown

Fix a race condition where sending data on DC fail as it doesn't check for the readyState to be opened before sending data onto the DC.

Fix a race condition where sending data on DC fail as it doesn't check for the readyState to be opened before sending data onto the DC.

@freddyaboulton freddyaboulton left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @gtherond !

@freddyaboulton

Copy link
Copy Markdown
Collaborator

Can you please fix the lint? Looks like the indentation does not line up

@gtherond

Copy link
Copy Markdown
Author

Can you please fix the lint? Looks like the indentation does not line up

For sure!

@gtherond

Copy link
Copy Markdown
Author

Ok, I've added the missing part of the patch, sorry it seems I didn't performed my diff patch properly the first time ^^

@gtherond

gtherond commented Mar 9, 2026

Copy link
Copy Markdown
Author

Any way to unlock the workflows approval?

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