Skip to content

Commit

Permalink
refactor(ext/websocket): align error messages
Browse files Browse the repository at this point in the history
Aligns the error messages in the ext/websocket folder to be in-line with
the Deno style guide.

denoland#25269
  • Loading branch information
irbull committed Sep 13, 2024
1 parent d162733 commit 588512a
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions ext/websocket/01_websocket.js
Original file line number Diff line number Diff line change
Expand Up @@ -161,14 +161,14 @@ class WebSocket extends EventTarget {

if (wsURL.protocol !== "ws:" && wsURL.protocol !== "wss:") {
throw new DOMException(
"Only ws & wss schemes are allowed in a WebSocket URL.",
`Only ws & wss schemes are allowed in a WebSocket URL: received ${wsURL.protocol}`,
"SyntaxError",
);
}

if (wsURL.hash !== "" || StringPrototypeEndsWith(wsURL.href, "#")) {
throw new DOMException(
"Fragments are not allowed in a WebSocket URL.",
"Fragments are not allowed in a WebSocket URL",
"SyntaxError",
);
}
Expand All @@ -195,7 +195,7 @@ class WebSocket extends EventTarget {
)
) {
throw new DOMException(
"Can't supply multiple times the same protocol.",
"Cannot supply multiple times the same protocol",
"SyntaxError",
);
}
Expand All @@ -208,7 +208,7 @@ class WebSocket extends EventTarget {
)
) {
throw new DOMException(
"Invalid protocol value.",
"Invalid protocol value",
"SyntaxError",
);
}
Expand Down Expand Up @@ -330,7 +330,7 @@ class WebSocket extends EventTarget {
data = webidl.converters.WebSocketSend(data, prefix, "Argument 1");

if (this[_readyState] !== OPEN) {
throw new DOMException("readyState not OPEN", "InvalidStateError");
throw new DOMException("'readyState' not OPEN", "InvalidStateError");
}

if (this[_sendQueue].length === 0) {
Expand Down Expand Up @@ -376,7 +376,7 @@ class WebSocket extends EventTarget {
!(code === 1000 || (3000 <= code && code < 5000))
) {
throw new DOMException(
"The close code must be either 1000 or in the range of 3000 to 4999.",
`The close code must be either 1000 or in the range of 3000 to 4999: received ${code}`,
"InvalidAccessError",
);
}
Expand All @@ -387,7 +387,7 @@ class WebSocket extends EventTarget {
TypedArrayPrototypeGetByteLength(core.encode(reason)) > 123
) {
throw new DOMException(
"The close reason may not be longer than 123 bytes.",
"The close reason may not be longer than 123 bytes",
"SyntaxError",
);
}
Expand Down

0 comments on commit 588512a

Please sign in to comment.