Skip to content

Long Polling disabled update#23033

Merged
guardrex merged 3 commits into
mainfrom
guardrex/blazor-websockets-only
Aug 19, 2021
Merged

Long Polling disabled update#23033
guardrex merged 3 commits into
mainfrom
guardrex/blazor-websockets-only

Conversation

@guardrex
Copy link
Copy Markdown
Collaborator

@guardrex guardrex commented Aug 16, 2021

Fixes #22863
Addresses #22045

  • Is (function start() { ... })() necessary in this scenario? AFAIK, we document calling Blazor.XXX directly in the <script> everywhere else.

@danroth27 ... I cross-link to the Azure Feedback item ...

Support WebSocket connections on Azure Front Door

... is that a good 👍 bikeshedding scenario (they should be aware of it and vote) or bad 👎 bikeshedding scenario (they'll need to find that on their own and let's chop 🔪 the cross-link here)?

cc: @mkArtakMSFT

Copy link
Copy Markdown
Contributor

@TanayParikh TanayParikh left a comment

Choose a reason for hiding this comment

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

Thanks!

Comment thread aspnetcore/blazor/fundamentals/signalr.md Outdated
Co-authored-by: Tanay Parikh <TanayParikh@users.noreply.github.com>
@TanayParikh
Copy link
Copy Markdown
Contributor

/ping @danroth27 regarding the Azure Feedback Item.

@guardrex
Copy link
Copy Markdown
Collaborator Author

I think he's busy 😄 ..... 🏃🏃🏃🏃😅.

If he (or Artak) doesn't respond within a day, I'll go ahead and merge this. I can always carve 🔪 that link out later.

@guardrex
Copy link
Copy Markdown
Collaborator Author

@TanayParikh ... I'll go ahead now, and we'll review that link later. I'm not too worried about the link as a bikeshedding event ... the whole point of receiving feedback is to allow devs to express their desire for a feature and their concerns about its implementation. I'm more concerned about keeping up with it because its a point-in-time item. I've made a note on my project tracking issue to look again early next year to see where they're at with that feature request and adjust the text in the Threat Mitigation topic accordingly.

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.

Using Long Polling for Blazor Server

2 participants