Skip to content

Commit

Permalink
BREAKING(unstable): Rename Deno.WebSocketUpgrade::websocket to socket (
Browse files Browse the repository at this point in the history
  • Loading branch information
nayeemrmn committed Jul 28, 2021
1 parent c198535 commit 42afbed
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 17 deletions.
10 changes: 5 additions & 5 deletions cli/tests/unit/http_test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -641,12 +641,12 @@ unitTest({ perms: { net: true } }, async function httpServerWebSocket() {
const { request, respondWith } = (await httpConn.nextRequest())!;
const {
response,
websocket,
socket,
} = Deno.upgradeWebSocket(request);
websocket.onerror = () => fail();
websocket.onmessage = (m) => {
websocket.send(m.data);
websocket.close();
socket.onerror = () => fail();
socket.onmessage = (m) => {
socket.send(m.data);
socket.close();
};
await respondWith(response);
break;
Expand Down
8 changes: 4 additions & 4 deletions extensions/http/01_http.js
Original file line number Diff line number Diff line change
Expand Up @@ -368,11 +368,11 @@

const response = fromInnerResponse(r, "immutable");

const websocket = webidl.createBranded(WebSocket);
setEventTargetData(websocket);
response[_ws] = websocket;
const socket = webidl.createBranded(WebSocket);
setEventTargetData(socket);
response[_ws] = socket;

return { response, websocket };
return { response, socket };
}

window.__bootstrap.http = {
Expand Down
16 changes: 8 additions & 8 deletions extensions/http/lib.deno_http.unstable.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ declare namespace Deno {

export interface WebSocketUpgrade {
response: Response;
websocket: WebSocket;
socket: WebSocket;
}

export interface UpgradeWebSocketOptions {
Expand All @@ -38,16 +38,16 @@ declare namespace Deno {
* const httpConn = Deno.serveHttp(conn);
* const e = await httpConn.nextRequest();
* if (e) {
* const { websocket, response } = Deno.upgradeWebSocket(e.request);
* websocket.onopen = () => {
* websocket.send("Hello World!");
* const { socket, response } = Deno.upgradeWebSocket(e.request);
* socket.onopen = () => {
* socket.send("Hello World!");
* };
* websocket.onmessage = (e) => {
* socket.onmessage = (e) => {
* console.log(e.data);
* websocket.close();
* socket.close();
* };
* websocket.onclose = () => console.log("WebSocket has been closed.");
* websocket.onerror = (e) => console.error("WebSocket error:", e.message);
* socket.onclose = () => console.log("WebSocket has been closed.");
* socket.onerror = (e) => console.error("WebSocket error:", e.message);
* e.respondWith(response);
* }
* ```
Expand Down

0 comments on commit 42afbed

Please sign in to comment.