Have PingInterval stop the connection on 401's and 403's. #2360

Closed
NTaylorMullen opened this Issue Jul 29, 2013 · 1 comment

2 participants

@NTaylorMullen

We should stop the SignalR connection on 401's and 403's from ping interval.

We should also restrict pings to only occur while the connection is in the connected state.

@NTaylorMullen NTaylorMullen added a commit that referenced this issue Jul 29, 2013
@NTaylorMullen NTaylorMullen Modified ping server to stop the connection in the case that the ping…
… fails due to a 401 or a 403 response.

- This involved adding a 3 new error messages to make everything consistent.

#2360
4a7dcc0
@NTaylorMullen NTaylorMullen added a commit that referenced this issue Jul 29, 2013
@NTaylorMullen NTaylorMullen Added tests to verify that 401 and 403 status code responses from the…
… server on pings will stop the connection.

- This involved creating a StatusCodeConnection that will change the response status code based on a query string variable AND will respond with a specified status code.

#2360
ce7c00c
@NTaylorMullen NTaylorMullen added a commit that referenced this issue Jul 30, 2013
@NTaylorMullen NTaylorMullen Modified ping server to stop the connection in the case that the ping…
… fails due to a 401 or a 403 response.

- This involved adding a 3 new error messages to make everything consistent.

#2360
fe42374
@NTaylorMullen NTaylorMullen added a commit that referenced this issue Jul 30, 2013
@NTaylorMullen NTaylorMullen Added tests to verify that 401 and 403 status code responses from the…
… server on pings will stop the connection.

- This involved creating a StatusCodeConnection that will change the response status code based on a query string variable AND will respond with a specified status code.

#2360
c5a00c0
@gustavo-armenta

tested with forms auth (302) redirects, fails to parse json, and stop connection. On basic auth it returns 401 and stop connection

@NTaylorMullen NTaylorMullen added a commit that referenced this issue Sep 30, 2013
@NTaylorMullen NTaylorMullen Added tests to verify that 401 and 403 status code responses from the…
… server on pings will stop the connection.

- This involved creating a StatusCodeConnection that will change the response status code based on a query string variable AND will respond with a specified status code.

#2360
9b4f7f3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment