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 5, 2023
1 parent 3bcd7bb commit d7bb039
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 d7bb039

Please sign in to comment.