[Modify] It throws exceptions
This commit is contained in:
parent
6c2f00d5e5
commit
4ce77fb5c5
@ -1127,12 +1127,52 @@ namespace WebSocketSharp.Server
|
||||
/// A <see cref="string"/> that represents the reason for the WebSocket
|
||||
/// connection close. The size must be 123 bytes or less.
|
||||
/// </param>
|
||||
/// <exception cref="ArgumentOutOfRangeException">
|
||||
/// The size of <paramref name="reason"/> is greater than 123 bytes.
|
||||
/// </exception>
|
||||
/// <exception cref="ArgumentException">
|
||||
/// <para>
|
||||
/// <paramref name="code"/> is
|
||||
/// <see cref="CloseStatusCode.MandatoryExtension"/>.
|
||||
/// </para>
|
||||
/// <para>
|
||||
/// -or-
|
||||
/// </para>
|
||||
/// <para>
|
||||
/// <paramref name="code"/> is
|
||||
/// <see cref="CloseStatusCode.NoStatus"/> and
|
||||
/// there is <paramref name="reason"/>.
|
||||
/// </para>
|
||||
/// <para>
|
||||
/// -or-
|
||||
/// </para>
|
||||
/// <para>
|
||||
/// <paramref name="reason"/> could not be UTF-8-encoded.
|
||||
/// </para>
|
||||
/// </exception>
|
||||
public void Stop (CloseStatusCode code, string reason)
|
||||
{
|
||||
string msg;
|
||||
if (!WebSocket.CheckParametersForClose (code, reason, false, out msg)) {
|
||||
_log.Error (msg);
|
||||
return;
|
||||
if (code == CloseStatusCode.MandatoryExtension) {
|
||||
var msg = "MandatoryExtension cannot be used.";
|
||||
throw new ArgumentException (msg, "code");
|
||||
}
|
||||
|
||||
if (!reason.IsNullOrEmpty ()) {
|
||||
if (code == CloseStatusCode.NoStatus) {
|
||||
var msg = "NoStatus cannot be used.";
|
||||
throw new ArgumentException (msg, "code");
|
||||
}
|
||||
|
||||
byte[] bytes;
|
||||
if (!reason.TryGetUTF8EncodedBytes (out bytes)) {
|
||||
var msg = "It could not be UTF-8-encoded.";
|
||||
throw new ArgumentException (msg, "reason");
|
||||
}
|
||||
|
||||
if (bytes.Length > 123) {
|
||||
var msg = "Its size is greater than 123 bytes.";
|
||||
throw new ArgumentOutOfRangeException ("reason", msg);
|
||||
}
|
||||
}
|
||||
|
||||
stop ((ushort) code, reason);
|
||||
|
Loading…
Reference in New Issue
Block a user