Add retries when initial connection fails #21711
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In this particular case, the client failed to connect with
Error: WebSocket closed with status code: 1006
. The recommendation here is to collect server logs. Unfortunately the linked build is old and deleted, so we no longer have any logs to look at.Given that the Blazor auth tests appear to continue running just fine and this test works locally, the change I have:
a) Adds some retries in case the browser is unable to establish the SignalR connection
b) Change the connection detection to be more robust than tossing in a delay.