MirrorMirror
/
Signal-Server
mirror of https://github.com/signalapp/Signal-Server.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Move `onErrorResume` to individual `sendMessage` `Mono`

This commit is contained in:
Chris Eager 2023-09-06 16:52:08 -05:00 committed by ravi-signal
parent 656326355a
commit a9a2e40fed
1 changed files with 11 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
service/src/main/java/org/whispersystems/textsecuregcm/websocket/WebSocketConnection.java
View File

@ -360,7 +360,7 @@ public class WebSocketConnection implements MessageAvailabilityListener, Displac
.limitRate(MESSAGE_PUBLISHER_LIMIT_RATE) .limitRate(MESSAGE_PUBLISHER_LIMIT_RATE)
.flatMapSequential(envelope -> .flatMapSequential(envelope ->
Mono.fromFuture(() -> sendMessage(envelope) Mono.fromFuture(() -> sendMessage(envelope)
.orTimeout(sendFuturesTimeoutMillis, TimeUnit.MILLISECONDS))) .orTimeout(sendFuturesTimeoutMillis, TimeUnit.MILLISECONDS))
.onErrorResume( .onErrorResume(
// let the first error pass through to terminate the subscription // let the first error pass through to terminate the subscription
e -> { e -> {
@ -372,6 +372,7 @@ public class WebSocketConnection implements MessageAvailabilityListener, Displac
// otherwise just emit nothing // otherwise just emit nothing
e -> Mono.empty() e -> Mono.empty()
) )
)
.subscribeOn(messageDeliveryScheduler) .subscribeOn(messageDeliveryScheduler)
.subscribe( .subscribe(
// no additional consumer of values - it is Flux<Void> by now // no additional consumer of values - it is Flux<Void> by now