Skip to content

Commit ea3328f

Browse files
committed
fix(server-*): rename message ping & pong body & type
1 parent f0af59e commit ea3328f

File tree

3 files changed

+16
-12
lines changed

3 files changed

+16
-12
lines changed

packages/server-runtime/src/index.ts

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import type {
99
import type { AuthenticatedPeer, Peer } from './types'
1010

1111
import { availableLogLevelStrings, Format, LogLevelString, logLevelStringToLogLevelMap, useLogg } from '@guiiai/logg'
12-
import { MessageHeartbeat, MessageHeartbeatMark, WebSocketEventSource } from '@proj-airi/server-shared/types'
12+
import { MessageHeartbeat, MessageHeartbeatKind, WebSocketEventSource } from '@proj-airi/server-shared/types'
1313
import { defineWebSocketHandler, H3 } from 'h3'
1414

1515
import { optionOrEnv } from './config'
@@ -48,6 +48,7 @@ export function setupApp(options?: {
4848
}
4949
heartbeat?: {
5050
readTimeout?: number
51+
message?: MessageHeartbeat | string
5152
}
5253
}): H3 {
5354
const authToken = optionOrEnv(options?.auth?.token, 'AUTHENTICATION_TOKEN', '')
@@ -67,6 +68,7 @@ export function setupApp(options?: {
6768
const peers = new Map<string, AuthenticatedPeer>()
6869
const peersByModule = new Map<string, Map<number | undefined, AuthenticatedPeer>>()
6970
const heartbeatTtlMs = options?.heartbeat?.readTimeout ?? DEFAULT_HEARTBEAT_TTL_MS
71+
const heartbeatMessage = options?.heartbeat?.message ?? MessageHeartbeat.Pong
7072
const routingMiddleware = [
7173
...(options?.routing?.policy ? [createPolicyMiddleware(options.routing.policy)] : []),
7274
...(options?.routing?.middleware ?? []),
@@ -168,12 +170,12 @@ export function setupApp(options?: {
168170
p.lastHeartbeatAt = Date.now()
169171
}
170172

171-
if (event.data.message === MessageHeartbeat.Ping) {
173+
if (event.data.kind === MessageHeartbeatKind.Ping) {
172174
send(peer, {
173175
type: 'transport:connection:heartbeat',
174176
data: {
175-
message: MessageHeartbeat.Pong,
176-
mark: MessageHeartbeatMark.Pong,
177+
kind: MessageHeartbeatKind.Pong,
178+
message: heartbeatMessage,
177179
at: Date.now(),
178180
},
179181
source: WebSocketEventSource.Server,

packages/server-sdk/src/client.ts

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,4 @@
11
import type {
2-
MessageHeartbeat,
3-
MessageHeartbeatMark,
42
MetadataEventSource,
53
WebSocketBaseEvent,
64
WebSocketEvent,
@@ -12,6 +10,10 @@ import type {
1210
import WebSocket from 'crossws/websocket'
1311

1412
import { sleep } from '@moeru/std'
13+
import {
14+
MessageHeartbeat,
15+
MessageHeartbeatKind,
16+
} from '@proj-airi/server-shared/types'
1517

1618
export interface ClientOptions<C = undefined> {
1719
url?: string
@@ -91,7 +93,7 @@ export class Client<C = undefined> {
9193
})
9294

9395
this.onEvent('transport:connection:heartbeat', (event) => {
94-
if (event.data.message === MessageHeartbeat.Ping) {
96+
if (event.data.kind === MessageHeartbeatKind.Ping) {
9597
this.sendHeartbeatPong()
9698
}
9799
})
@@ -358,8 +360,8 @@ export class Client<C = undefined> {
358360
this.send({
359361
type: 'transport:connection:heartbeat',
360362
data: {
363+
kind: MessageHeartbeatKind.Ping,
361364
message: this.opts.heartbeat?.message ?? MessageHeartbeat.Ping,
362-
mark: MessageHeartbeatMark.Ping,
363365
at: Date.now(),
364366
},
365367
})
@@ -370,8 +372,8 @@ export class Client<C = undefined> {
370372
this.send({
371373
type: 'transport:connection:heartbeat',
372374
data: {
375+
kind: MessageHeartbeatKind.Pong,
373376
message: MessageHeartbeat.Pong,
374-
mark: MessageHeartbeatMark.Pong,
375377
at: Date.now(),
376378
},
377379
})

packages/server-shared/src/types/websocket/events.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,12 +35,12 @@ export interface RouteConfig {
3535
bypass?: boolean
3636
}
3737

38-
export enum MessageHeartbeat {
38+
export enum MessageHeartbeatKind {
3939
Ping = 'ping',
4040
Pong = 'pong',
4141
}
4242

43-
export enum MessageHeartbeatMark {
43+
export enum MessageHeartbeat {
4444
Ping = '🩵',
4545
Pong = '💛',
4646
}
@@ -287,8 +287,8 @@ export interface WebSocketEvents<C = undefined> {
287287
}
288288

289289
'transport:connection:heartbeat': {
290+
kind: MessageHeartbeatKind
290291
message: MessageHeartbeat | string
291-
mark?: '🩵' | '💛'
292292
at?: number
293293
}
294294

0 commit comments

Comments
 (0)