Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

start-stop-start long polling results on a stream of poll requests #2160

Closed
gustavo-armenta opened this Issue Jun 18, 2013 · 3 comments

Comments

Projects
None yet
5 participants
Contributor

gustavo-armenta commented Jun 18, 2013

Modify samples\Microsoft.AspNet.SignalR.Samples\Hubs\Test\chat.js to to this:

    hubConnection.start({ transport: activeTransport });
    hubConnection.stop();
    hubConnection.start({ transport: activeTransport });

On browser, open JS Console and connect to http://localhost:40476/hubs/test/chat.aspx?transport=longPolling

You will see infinite attempts to do a poll request

Owner

davidfowl commented Jun 18, 2013

Is this a regression?

Contributor

gustavo-armenta commented Jun 18, 2013

Yes, it's a regression.

@NTaylorMullen NTaylorMullen added a commit that referenced this issue Jun 20, 2013

@NTaylorMullen NTaylorMullen Added tracking for the negotiate request so calling stop properly abo…
…rts the request.

- Without aborting the negotiate request starting and stopping in quick succession can cause multiple active ajax requests attempting to connect.

#2160
2925298

@NTaylorMullen NTaylorMullen added a commit that referenced this issue Jun 20, 2013

@NTaylorMullen NTaylorMullen Added test to verify starting and stopping doesn't result in multiple…
… client side connections.

- Also had to fix up some past tests that were not returning their ajax handles in replaced ajax functions.

#2160
fa97122

@NTaylorMullen NTaylorMullen added a commit that referenced this issue Jun 21, 2013

@NTaylorMullen @NTaylorMullen NTaylorMullen + NTaylorMullen Added tracking for the negotiate request so calling stop properly abo…
…rts the request.

- Without aborting the negotiate request starting and stopping in quick succession can cause multiple active ajax requests attempting to connect.

#2160
f95f940

@NTaylorMullen NTaylorMullen added a commit that referenced this issue Jun 21, 2013

@NTaylorMullen @NTaylorMullen NTaylorMullen + NTaylorMullen Added test to verify starting and stopping doesn't result in multiple…
… client side connections.

- Also had to fix up some past tests that were not returning their ajax handles in replaced ajax functions.

#2160
fd91c95

@NTaylorMullen NTaylorMullen added a commit that referenced this issue Jun 21, 2013

@NTaylorMullen NTaylorMullen Added test to verify hub event handlers are only triggered once per b…
…inding.

- This is a multi-purpose test to verify 3 bugs.

#2187
#2160
#2190
120eb62

@NTaylorMullen NTaylorMullen added a commit that referenced this issue Jun 22, 2013

@NTaylorMullen @NTaylorMullen NTaylorMullen + NTaylorMullen Added tracking for the negotiate request so calling stop properly abo…
…rts the request.

- Without aborting the negotiate request starting and stopping in quick succession can cause multiple active ajax requests attempting to connect.

#2160
98d6464

@NTaylorMullen NTaylorMullen added a commit that referenced this issue Jun 22, 2013

@NTaylorMullen @NTaylorMullen NTaylorMullen + NTaylorMullen Added test to verify starting and stopping doesn't result in multiple…
… client side connections.

- Also had to fix up some past tests that were not returning their ajax handles in replaced ajax functions.

#2160
9728e88

@NTaylorMullen NTaylorMullen added a commit that referenced this issue Jun 22, 2013

@NTaylorMullen NTaylorMullen Added test to verify hub event handlers are only triggered once per b…
…inding.

- This is a multi-purpose test to verify 3 bugs.

#2187
#2160
#2190
e4cf8c6

@ghost ghost assigned muratg and Xiaohongt Jun 22, 2013

Contributor

Xiaohongt commented Jun 23, 2013

verified

@Xiaohongt Xiaohongt closed this Jun 23, 2013

@NTaylorMullen NTaylorMullen added a commit that referenced this issue Oct 2, 2013

@NTaylorMullen @halter73 NTaylorMullen + halter73 Added test to verify hub event handlers are only triggered once per b…
…inding.

- This is a multi-purpose test to verify 3 bugs.

#2187
#2160
#2190
0e73994

@NTaylorMullen NTaylorMullen added a commit that referenced this issue Oct 2, 2013

@NTaylorMullen @halter73 NTaylorMullen + halter73 Added test to verify hub event handlers are only triggered once per b…
…inding.

- This is a multi-purpose test to verify 3 bugs.

#2187
#2160
#2190
72de4d8

@NTaylorMullen NTaylorMullen added a commit that referenced this issue Oct 7, 2013

@NTaylorMullen @halter73 NTaylorMullen + halter73 Added test to verify hub event handlers are only triggered once per b…
…inding.

- This is a multi-purpose test to verify 3 bugs.

#2187
#2160
#2190
33c2693

@NTaylorMullen NTaylorMullen added a commit that referenced this issue Oct 7, 2013

@NTaylorMullen @halter73 NTaylorMullen + halter73 Added test to verify hub event handlers are only triggered once per b…
…inding.

- This is a multi-purpose test to verify 3 bugs.

#2187
#2160
#2190
6a03c89
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment