Remove legacy `DisconnectRequested` protobuf entity
This commit is contained in:
parent
2f890f7bb3
commit
6845ba9b90
|
@ -333,10 +333,6 @@ public class WebSocketConnectionEventManager extends RedisClusterPubSubAdapter<b
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
case DISCONNECT_REQUESTED -> {
|
|
||||||
// Handle events from `DisconnectionRequestManager` instead
|
|
||||||
}
|
|
||||||
|
|
||||||
case MESSAGES_PERSISTED -> listenerEventExecutor.execute(listener::handleMessagesPersisted);
|
case MESSAGES_PERSISTED -> listenerEventExecutor.execute(listener::handleMessagesPersisted);
|
||||||
|
|
||||||
default -> logger.warn("Unexpected client event type: {}", clientEvent.getClass());
|
default -> logger.warn("Unexpected client event type: {}", clientEvent.getClass());
|
||||||
|
|
|
@ -10,10 +10,10 @@ option java_package = "org.whispersystems.textsecuregcm.push";
|
||||||
option java_multiple_files = true;
|
option java_multiple_files = true;
|
||||||
|
|
||||||
message ClientEvent {
|
message ClientEvent {
|
||||||
|
reserved 3;
|
||||||
oneof event {
|
oneof event {
|
||||||
NewMessageAvailableEvent new_message_available = 1;
|
NewMessageAvailableEvent new_message_available = 1;
|
||||||
ClientConnectedEvent client_connected = 2;
|
ClientConnectedEvent client_connected = 2;
|
||||||
DisconnectRequested disconnect_requested = 3;
|
|
||||||
MessagesPersistedEvent messages_persisted = 4;
|
MessagesPersistedEvent messages_persisted = 4;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -31,13 +31,6 @@ message ClientConnectedEvent {
|
||||||
bytes server_id = 1;
|
bytes server_id = 1;
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* Indicates that the server has requested that the client disconnect due to
|
|
||||||
* (for example) account lifecycle events.
|
|
||||||
*/
|
|
||||||
message DisconnectRequested {
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Indicates that messages for the client have been persisted from short-term
|
* Indicates that messages for the client have been persisted from short-term
|
||||||
* storage to long-term storage.
|
* storage to long-term storage.
|
Loading…
Reference in New Issue