Skip to content

Commit f5c6484

Browse files
author
Nickolay Savchenko
committed
Fixed heartbeat
1 parent 2e4f12e commit f5c6484

File tree

3 files changed

+14
-10
lines changed

3 files changed

+14
-10
lines changed

example-client/src/main/java/ua/naiksoftware/stompclientexample/MainActivity.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ public void connectStomp(View view) {
9898
}
9999
});
100100

101-
// compositeDisposable.add(dispLifecycle);
101+
compositeDisposable.add(dispLifecycle);
102102

103103
// Receive greetings
104104
Disposable dispTopic = mStompClient.topic("/topic/greetings")
@@ -111,7 +111,7 @@ public void connectStomp(View view) {
111111
Log.e(TAG, "Error on subscribe topic", throwable);
112112
});
113113

114-
// compositeDisposable.add(dispTopic);
114+
compositeDisposable.add(dispTopic);
115115

116116
mStompClient.connect(headers);
117117
}

lib/src/main/java/ua/naiksoftware/stomp/HeartBeatTask.java

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,9 @@ public class HeartBeatTask {
2121
private int serverHeartbeat = 0;
2222
private int clientHeartbeat = 0;
2323

24+
private int serverHeartbeatNew = 0;
25+
private int clientHeartbeatNew = 0;
26+
2427
private transient long lastServerHeartBeat = 0;
2528

2629
private transient Disposable clientSendHeartBeatTask;
@@ -35,19 +38,19 @@ public HeartBeatTask(SendCallback sendCallback, @Nullable FailedListener failedL
3538
}
3639

3740
public void setServerHeartbeat(int serverHeartbeat) {
38-
this.serverHeartbeat = serverHeartbeat;
41+
this.serverHeartbeatNew = serverHeartbeat;
3942
}
4043

4144
public void setClientHeartbeat(int clientHeartbeat) {
42-
this.clientHeartbeat = clientHeartbeat;
45+
this.clientHeartbeatNew = clientHeartbeat;
4346
}
4447

4548
public int getServerHeartbeat() {
46-
return serverHeartbeat;
49+
return serverHeartbeatNew;
4750
}
4851

4952
public int getClientHeartbeat() {
50-
return clientHeartbeat;
53+
return clientHeartbeatNew;
5154
}
5255

5356
public boolean consumeHeartBeat(StompMessage message) {
@@ -96,13 +99,13 @@ private void heartBeatHandshake(final String heartBeatHeader) {
9699
if (heartBeatHeader != null) {
97100
// The heart-beat header is OPTIONAL
98101
final String[] heartbeats = heartBeatHeader.split(",");
99-
if (clientHeartbeat > 0) {
102+
if (clientHeartbeatNew > 0) {
100103
//there will be heart-beats every MAX(<cx>,<sy>) milliseconds
101-
clientHeartbeat = Math.max(clientHeartbeat, Integer.parseInt(heartbeats[1]));
104+
clientHeartbeat = Math.max(clientHeartbeatNew, Integer.parseInt(heartbeats[1]));
102105
}
103-
if (serverHeartbeat > 0) {
106+
if (serverHeartbeatNew > 0) {
104107
//there will be heart-beats every MAX(<cx>,<sy>) milliseconds
105-
serverHeartbeat = Math.max(serverHeartbeat, Integer.parseInt(heartbeats[0]));
108+
serverHeartbeat = Math.max(serverHeartbeatNew, Integer.parseInt(heartbeats[0]));
106109
}
107110
}
108111
if (clientHeartbeat > 0 || serverHeartbeat > 0) {

lib/src/main/java/ua/naiksoftware/stomp/StompClient.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -119,6 +119,7 @@ public void connect(@Nullable List<StompHeader> _headers) {
119119

120120
connectionProvider.send(new StompMessage(StompCommand.CONNECT, headers, null).compile(legacyWhitespace))
121121
.subscribe(() -> {
122+
Log.d(TAG, "Publish open");
122123
lifecyclePublishSubject.onNext(lifecycleEvent);
123124
});
124125
break;

0 commit comments

Comments
 (0)