huahaofeng
/
aspnetcore
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix ANCM websocket disconnect issue

This commit is contained in:
Justin Kotalik 2018-11-15 23:25:16 +00:00
parent 88273412b8 1858527fb0
commit f7633d8be9
1 changed files with 23 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
src/IISIntegration/src/AspNetCoreModuleV1/AspNetCore/src/websockethandler.cxx
View File

@ -266,6 +266,7 @@ Routine Description:
HRESULT hr = S_OK;
//DWORD dwBuffSize = RECEIVE_BUFFER_SIZE;
*fHandleCreated = FALSE;
_pHandler = pHandler;
EnterCriticalSection(&_RequestLock);
@ -1145,21 +1146,34 @@ Arguments:
_fCleanupInProgress = TRUE;
_fIndicateCompletionToIis = TRUE;
//
// We need cancel IO for fast error handling
// Reivist the code once CanelOutstandingIO api is available
//
/*if (_pWebSocketContext != NULL)
if (reason == ClientDisconnect || reason == ServerStateUnavailable)
{
_pWebSocketContext->CancelOutstandingIO();
}*/
_pHttpContext->CancelIo();
//
// Calling shutdown to notify the backend about disonnect
//
WINHTTP_HELPER::sm_pfnWinHttpWebSocketShutdown(
_hWebSocketRequest,
WINHTTP_WEB_SOCKET_SERVER_ERROR_CLOSE_STATUS, // indicate that a server is terminating the connection because it encountered
// an unexpected condition that prevent it from fulfilling the request
NULL, // Reason
0); // length og Reason
}
if (reason == ServerDisconnect || reason == ServerStateUnavailable)
{
_pHttpContext->CancelIo();
//
// CancelIo sometime may not be able to cannel pending websocket IO
// ResetConnection to force IISWebsocket module to release the pipeline
//
_pHttpContext->GetResponse()->ResetConnection();
}
//
// Don't close the handle here,
// as it trigger a WinHttp callback and let IIS pipeline continue
// Handle should be closed only in IndicateCompletionToIIS
IndicateCompletionToIIS();
Finished:
if (fLocked)
{