fix: Do not retry connection if data is too large (#586)

* Do not retry connection if data is too large * lint * Remove yarn.lock
ueberdosis · Apr 23, 2023 · 462a87f · 462a87f
1 parent 7c38bd7
commit 462a87f
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/packages/common/src/CloseEvents.ts b/packages/common/src/CloseEvents.ts
@@ -3,6  3,16 @@ export interface CloseEvent {
   reason: string,
 }
 
 /**
  * The server is terminating the connection because a data frame was received
  * that is too large.
  * See: https://developer.mozilla.org/en-US/docs/Web/API/CloseEvent/code
  */
 export const MessageTooBig: CloseEvent = {
   code: 1009,
   reason: 'Message Too Big',
 }
 
 /**
  * The server successfully processed the request, asks that the requester reset
  * its document view, and is not returning any content.

diff --git a/packages/provider/src/HocuspocusProviderWebsocket.ts b/packages/provider/src/HocuspocusProviderWebsocket.ts
@@ -4,7  4,7 @@ import * as url from 'lib0/url'
 import type { MessageEvent } from 'ws'
 import { retry } from '@lifeomic/attempt'
 import {
-  Forbidden, Unauthorized, WsReadyStates,
   Forbidden, MessageTooBig, Unauthorized, WsReadyStates,
 } from '@hocuspocus/common'
 import { Event } from 'ws'
 import EventEmitter from './EventEmitter'
@@ -425,6  425,11 @@ export class HocuspocusProviderWebsocket extends EventEmitter {
       }
     }
 
     if (event.code === MessageTooBig.code) {
       console.warn(`[HocuspocusProvider] Connection closed with status MessageTooBig: ${event.reason}`)
       this.shouldConnect = false
     }
 
     if (this.connectionAttempt) {
       // That connection attempt failed.
       this.rejectConnectionAttempt()