Commit 49ef74cf authored by Finn Herzfeld's avatar Finn Herzfeld 🌵

minor cleanup, likely does not address session issues

* save account after disconnecting, just in case. Based on an unconfirmed suspicion that some data isn't getting saved
* improve handling of IOExceptions from manager.receiveMessages() - should help eliminate one of the more common exceptions that aren't actually problems
* remove some unused code
parent 72ef30e8
Pipeline #4067 failed with stages
in 6 minutes and 48 seconds
......@@ -1111,6 +1111,7 @@ class Manager {
messagePipe.shutdown();
messagePipe = null;
}
accountData.save();
}
}
......
......@@ -65,7 +65,9 @@ class MessageReceiver implements Manager.ReceiveMessageHandler, Runnable {
try {
manager.receiveMessages((long) (timeout * 1000), TimeUnit.MILLISECONDS, returnOnTimeout, ignoreAttachments, this);
} catch (IOException e) {
logger.debug("probably harmless IOException while receiving messages:", e);
if(sockets.size() > 0) {
throw e;
}
} catch (AssertionError e) {
logger.catching(e);
}
......
......@@ -120,10 +120,6 @@ public class AccountData {
}
public void save() throws IOException {
save(false);
}
public void save(boolean allowBlankPassword) throws IOException {
validate();
ObjectWriter writer = JSONUtil.GetWriter();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment