Skip to content

Commit

Permalink
feat: use globally available WebSocket client in runtimes that have it (
Browse files Browse the repository at this point in the history
  • Loading branch information
vladfrangu authored Dec 10, 2023
1 parent 8ace6fa commit 319ef9a
Showing 1 changed file with 4 additions and 6 deletions.
10 changes: 4 additions & 6 deletions packages/ws/src/ws/WebSocketShard.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ import { URLSearchParams } from 'node:url';
import { TextDecoder } from 'node:util';
import { inflate } from 'node:zlib';
import { Collection } from '@discordjs/collection';
import { lazy } from '@discordjs/util';
import { lazy, shouldUseGlobalFetchAndWebSocket } from '@discordjs/util';
import { AsyncQueue } from '@sapphire/async-queue';
import { AsyncEventEmitter } from '@vladfrangu/async_event_emitter';
import {
Expand Down Expand Up @@ -80,11 +80,9 @@ export interface SendRateLimitState {
resetAt: number;
}

// TODO(vladfrangu): enable this once https://github.com/oven-sh/bun/issues/3392 is solved
// const WebSocketConstructor: typeof WebSocket = shouldUseGlobalFetchAndWebSocket()
// ? (globalThis as any).WebSocket
// : WebSocket;
const WebSocketConstructor: typeof WebSocket = WebSocket;
const WebSocketConstructor: typeof WebSocket = shouldUseGlobalFetchAndWebSocket()
? (globalThis as any).WebSocket
: WebSocket;

export class WebSocketShard extends AsyncEventEmitter<WebSocketShardEventsMap> {
private connection: WebSocket | null = null;
Expand Down

0 comments on commit 319ef9a

Please sign in to comment.