Skip to content

CircuitHost should handle SemaphoreSlim correctly #11686

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
Closed
pranavkm opened this issue Jun 28, 2019 · 0 comments
Closed

CircuitHost should handle SemaphoreSlim correctly #11686

pranavkm opened this issue Jun 28, 2019 · 0 comments
Assignees
@ajaybhargavb
Labels
area-blazor Includes: Blazor, Razor Components bug This issue describes a behavior which is not expected - a bug. Done This issue has been fixed
Milestone
3.0.0-preview9

Comments

@pranavkm
Copy link
Contributor

@BrennanConroy pointed out we're not observing the results of WaitAsync:

https://github.com/aspnet/AspNetCore/blob/master/src/Components/Server/src/Circuits/CircuitHost.cs#L219
@pranavkm pranavkm added bug This issue describes a behavior which is not expected - a bug. area-blazor Includes: Blazor, Razor Components labels Jun 28, 2019
@mkArtakMSFT mkArtakMSFT added this to the 3.0.0-preview9 milestone Jul 2, 2019
@mkArtakMSFT mkArtakMSFT assigned ajaybhargavb and unassigned pranavkm Jul 24, 2019
@ajaybhargavb ajaybhargavb mentioned this issue Aug 2, 2019
Merged
@ajaybhargavb ajaybhargavb added Done This issue has been fixed and removed Working labels Aug 14, 2019
@ghost ghost locked as resolved and limited conversation to collaborators Dec 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ajaybhargavb ajaybhargavb

Labels
area-blazor Includes: Blazor, Razor Components bug This issue describes a behavior which is not expected - a bug. Done This issue has been fixed
Projects
None yet
Milestone
3.0.0-preview9
Development

No branches or pull requests
3 participants
@pranavkm @ajaybhargavb @mkArtakMSFT