Refactor ping mechanism in WalletWebSocketService for improved connection stability
- Updated the ping method to send a simple text message instead of using the unsupported ping method in browser WebSocket implementations. - Enhanced logging to indicate potential connection instability when ping fails. These changes improve the reliability of the WebSocket connection management, contributing to a more stable user experience.
This commit is contained in:
parent
15a7540515
commit
37a07c0c12
1 changed files with 4 additions and 3 deletions
|
|
@ -406,10 +406,11 @@ export class WalletWebSocketService extends BaseService {
|
|||
private sendPing(): void {
|
||||
if (this.ws && this.ws.readyState === WebSocket.OPEN) {
|
||||
try {
|
||||
// Send a ping frame (most WebSocket implementations support this)
|
||||
this.ws.ping?.()
|
||||
// Send a simple ping message (browser WebSocket doesn't have ping method)
|
||||
// Most WebSocket servers respond to text messages as keepalive
|
||||
this.ws.send(JSON.stringify({ type: 'ping' }))
|
||||
} catch (error) {
|
||||
console.log('WalletWebSocketService: Ping not supported, connection seems stable')
|
||||
console.log('WalletWebSocketService: Ping failed, connection may be unstable')
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue