Skip to content

Commit

Permalink
Remove redundant setting of useBinaryProtocol in options normalisation
Browse files Browse the repository at this point in the history
The property is already manipulated earlier in this method. Duplication
introduced in 4f66064; seems unintentional.
  • Loading branch information
lawrence-forooghian committed Sep 14, 2023
1 parent 8954827 commit 6bfe09b
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions src/common/lib/util/defaults.ts
Original file line number Diff line number Diff line change
Expand Up @@ -250,10 +250,6 @@ export function normaliseOptions(options: InternalClientOptions): NormalisedClie

return {
...options,
useBinaryProtocol:
'useBinaryProtocol' in options
? Platform.Config.supportsBinary && options.useBinaryProtocol
: Platform.Config.preferBinary,
realtimeHost,
restHost,
maxMessageSize: options.internal?.maxMessageSize || Defaults.maxMessageSize,
Expand Down

0 comments on commit 6bfe09b

Please sign in to comment.