Prevent empty username

diff --git a/lib/src/main/java/org/asamk/signal/manager/storage/SignalAccount.java b/lib/src/main/java/org/asamk/signal/manager/storage/SignalAccount.java
index a493f0d1905cd7961b7c1f771283d8dd35362350..762e95ce4912a7e980a6606c37bee9259424d07f 100644 (file)
--- a/lib/src/main/java/org/asamk/signal/manager/storage/SignalAccount.java
+++ b/lib/src/main/java/org/asamk/signal/manager/storage/SignalAccount.java
@@ -473,6 +473,9 @@ public class SignalAccount implements Closeable {
             registered = storage.registered;
             number = storage.number;
             username = storage.username;
             registered = storage.registered;
             number = storage.number;
             username = storage.username;

+            if ("".equals(username)) {
+                username = null;
+            }

             encryptedDeviceName = storage.encryptedDeviceName;
             deviceId = storage.deviceId;
             isMultiDevice = storage.isMultiDevice;
             encryptedDeviceName = storage.encryptedDeviceName;
             deviceId = storage.deviceId;
             isMultiDevice = storage.isMultiDevice;


@@ -549,6 +552,9 @@ public class SignalAccount implements Closeable {
         registered = Utils.getNotNullNode(rootNode, "registered").asBoolean();
         if (rootNode.hasNonNull("usernameIdentifier")) {
             username = rootNode.get("usernameIdentifier").asText();
         registered = Utils.getNotNullNode(rootNode, "registered").asBoolean();
         if (rootNode.hasNonNull("usernameIdentifier")) {
             username = rootNode.get("usernameIdentifier").asText();

+            if ("".equals(username)) {
+                username = null;
+            }

         }
         if (rootNode.hasNonNull("uuid")) {
             try {
         }
         if (rootNode.hasNonNull("uuid")) {
             try {

diff --git a/lib/src/main/java/org/asamk/signal/manager/syncStorage/AccountRecordProcessor.java b/lib/src/main/java/org/asamk/signal/manager/syncStorage/AccountRecordProcessor.java
index 9250505c5b09f45f13a221af12a620ce55f488e1..0e1a02b74298a5a7028f537cddc3c3c360dc08d0 100644 (file)
--- a/lib/src/main/java/org/asamk/signal/manager/syncStorage/AccountRecordProcessor.java
+++ b/lib/src/main/java/org/asamk/signal/manager/syncStorage/AccountRecordProcessor.java
@@ -104,7 +104,7 @@ public class AccountRecordProcessor extends DefaultStorageRecordProcessor<Signal
                 || local.hasSeenGroupStoryEducationSheet();
         final var username = remote.getUsername() != null && !remote.getUsername().isEmpty()
                 ? remote.getUsername()
                 || local.hasSeenGroupStoryEducationSheet();
         final var username = remote.getUsername() != null && !remote.getUsername().isEmpty()
                 ? remote.getUsername()

-                : local.getUsername() != null && !local.getUsername().isEmpty() ? local.getUsername() : null;
+                : local.getUsername();

         final var usernameLink = remote.getUsernameLink() != null ? remote.getUsernameLink() : local.getUsernameLink();
 
         final var mergedBuilder = new SignalAccountRecord.Builder(remote.getId().getRaw(), unknownFields).setGivenName(
         final var usernameLink = remote.getUsernameLink() != null ? remote.getUsernameLink() : local.getUsernameLink();
 
         final var mergedBuilder = new SignalAccountRecord.Builder(remote.getId().getRaw(), unknownFields).setGivenName(