Skip to content
This repository was archived by the owner on Jun 21, 2023. It is now read-only.

Commit 48a6c00

Browse files
authored
Merge pull request #148 from scalecube/fix-cancel-signal-on-client
Fix sending cancel signal on client
2 parents 1a100cf + 42a14f6 commit 48a6c00

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

services-gateway-client-transport/src/main/java/io/scalecube/services/gateway/transport/websocket/WebsocketGatewayClient.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ public Mono<ServiceMessage> requestResponse(ServiceMessage request) {
9090
.send(encodeRequest(request, sid), sid)
9191
.doOnSubscribe(s -> LOGGER.debug("Sending request {}", request))
9292
.then(session.newMonoProcessor(sid))
93-
.doOnCancel(() -> handleCancel(sid, session))
93+
.doOnCancel(() -> handleCancel(sid, request.qualifier(), session))
9494
.doFinally(s -> session.removeProcessor(sid)));
9595
});
9696
}
@@ -107,7 +107,7 @@ public Flux<ServiceMessage> requestStream(ServiceMessage request) {
107107
.send(encodeRequest(request, sid), sid)
108108
.doOnSubscribe(s -> LOGGER.debug("Sending request {}", request))
109109
.thenMany(session.newUnicastProcessor(sid))
110-
.doOnCancel(() -> handleCancel(sid, session))
110+
.doOnCancel(() -> handleCancel(sid, request.qualifier(), session))
111111
.doFinally(s -> session.removeProcessor(sid)));
112112
});
113113
}
@@ -211,10 +211,11 @@ private void onReadIdle(Connection connection) {
211211
.subscribe(null, ex -> LOGGER.warn("Can't send keepalive on readIdle: " + ex));
212212
}
213213

214-
private void handleCancel(long sid, WebsocketGatewayClientSession session) {
214+
private void handleCancel(long sid, String qualifier, WebsocketGatewayClientSession session) {
215215
ByteBuf byteBuf =
216216
codec.encode(
217217
ServiceMessage.builder()
218+
.qualifier(qualifier)
218219
.header(STREAM_ID, sid)
219220
.header(SIGNAL, Signal.CANCEL.codeAsString())
220221
.build());

0 commit comments

Comments
 (0)