Skip to content

[Blazor] Guidance on limiting circuits per user #11966

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.

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

Closed
javiercn opened this issue Jul 8, 2019 · 2 comments
Closed

[Blazor] Guidance on limiting circuits per user #11966

javiercn opened this issue Jul 8, 2019 · 2 comments
Assignees
Labels
area-blazor Includes: Blazor, Razor Components Docs This issue tracks updating documentation Done This issue has been fixed
Milestone

Comments

@javiercn
Copy link
Member

javiercn commented Jul 8, 2019

We don't have any mechanism that out of the box limits the number of circuits a user/ip can create, but this is achievable through current extensibility.

We should provide a sample that showcases this.

This can be achieved by keeping track of the endpoints that create circuits (for example through a handler on OnCircuitOpened) and capturing information (like the user principal or the client ip address) and using that to limit the connections per server instance.

@mkArtakMSFT mkArtakMSFT added the area-blazor Includes: Blazor, Razor Components label Jul 8, 2019
@mkArtakMSFT mkArtakMSFT added this to the 3.1.0 milestone Jul 8, 2019
@mkArtakMSFT mkArtakMSFT added Docs This issue tracks updating documentation cost: XS labels Jul 8, 2019
@mkArtakMSFT mkArtakMSFT modified the milestones: 3.1.0, 3.0.0-preview9 Jul 8, 2019
@mkArtakMSFT mkArtakMSFT modified the milestones: 3.0.0-preview9, 3.0.0, 3.0.0-rc1 Jul 12, 2019
@chrdlx
Copy link

chrdlx commented Jul 18, 2019

What about clients that share the same internet IP ? Which is the typical case for companies that use NAT to give inside users internet access.

@danroth27 danroth27 modified the milestones: 3.0.0-rc1, 3.0.0 Aug 19, 2019
@javiercn
Copy link
Member Author

@javiercn javiercn added the Done This issue has been fixed label Aug 22, 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.
Labels
area-blazor Includes: Blazor, Razor Components Docs This issue tracks updating documentation Done This issue has been fixed
Projects
None yet
Development

No branches or pull requests

4 participants