[Modify] Polish them

This commit is contained in:
sta 2018-04-27 19:21:56 +09:00
parent 1af3b26cb9
commit c0398c5bc1
2 changed files with 6 additions and 14 deletions

View File

@ -438,14 +438,6 @@ namespace WebSocketSharp.Net.WebSockets
Func<IIdentity, NetworkCredential> credentialsFinder Func<IIdentity, NetworkCredential> credentialsFinder
) )
{ {
if (scheme == AuthenticationSchemes.Anonymous)
return true;
if (scheme == AuthenticationSchemes.None) {
Close (HttpStatusCode.Forbidden);
return false;
}
var chal = new AuthenticationChallenge (scheme, realm).ToString (); var chal = new AuthenticationChallenge (scheme, realm).ToString ();
var retry = -1; var retry = -1;
@ -453,10 +445,8 @@ namespace WebSocketSharp.Net.WebSockets
auth = auth =
() => { () => {
retry++; retry++;
if (retry > 99) { if (retry > 99)
Close (HttpStatusCode.Forbidden);
return false; return false;
}
var user = HttpUtility.CreateUser ( var user = HttpUtility.CreateUser (
_request.Headers["Authorization"], _request.Headers["Authorization"],

View File

@ -806,6 +806,11 @@ namespace WebSocketSharp.Server
private void processRequest (TcpListenerWebSocketContext context) private void processRequest (TcpListenerWebSocketContext context)
{ {
if (!authenticateClient (context)) {
context.Close (HttpStatusCode.Forbidden);
return;
}
var uri = context.RequestUri; var uri = context.RequestUri;
if (uri == null) { if (uri == null) {
context.Close (HttpStatusCode.BadRequest); context.Close (HttpStatusCode.BadRequest);
@ -846,9 +851,6 @@ namespace WebSocketSharp.Server
cl, null, _secure, _sslConfigInUse, _log cl, null, _secure, _sslConfigInUse, _log
); );
if (!ctx.Authenticate (_authSchemes, _realmInUse, _userCredFinder))
return;
processRequest (ctx); processRequest (ctx);
} }
catch (Exception ex) { catch (Exception ex) {