Skip to content

Commit

Permalink
Move call to getMessageHandler().onResynchronize(); to right place
Browse files Browse the repository at this point in the history
#12043 changed resync message sending to be deferred to queue. Now also the setting of the semaphor in message handler needs to be deferred to its right place. Otherwise there is possibility for a timing glitch. I.e. MessageHandler is set to resync handling mode before message is actually send.

Fixes: #12151
  • Loading branch information
TatuLund authored Jan 5, 2021
1 parent a250bde commit 28687da
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -127,6 +127,7 @@ private void doSendInvocationsToServer() {
}
if (resynchronizeRequested) {
getLogger().info("Resynchronizing from server");
getMessageHandler().onResynchronize();
extraJson.put(ApplicationConstants.RESYNCHRONIZE_ID, true);
resynchronizeRequested = false;
}
Expand Down Expand Up @@ -356,7 +357,6 @@ private VLoadingIndicator getLoadingIndicator() {
* state from the server
*/
public void resynchronize() {
getMessageHandler().onResynchronize();
getLogger().info("Resynchronize from server requested");
resynchronizeRequested = true;
sendInvocationsToServer();
Expand Down

0 comments on commit 28687da

Please sign in to comment.