Remove dead code and fix inspection issues

lib/src/main/java/org/asamk/signal/manager/config/ServiceConfig.java

@@ -55,10 +55,6 @@ public class ServiceConfig {
         }
     }
 
-    public static AccountAttributes.Capabilities getCapabilities() {
-        return capabilities;
-    }
-
     public static KeyStore getIasKeyStore() {
         return iasKeyStore;
     }

lib/src/main/java/org/asamk/signal/manager/groups/GroupInviteLinkUrl.java

@@ -26,10 +26,6 @@ public final class GroupInviteLinkUrl {
                 GroupLinkPassword.fromBytes(group.getInviteLinkPassword().toByteArray()));
     }
 
-    public static boolean isGroupLink(String urlString) {
-        return getGroupUrl(urlString) != null;
-    }
-
     /**
      * @return null iff not a group url.
      * @throws InvalidGroupLinkException If group url, but cannot be parsed.
@@ -95,7 +91,7 @@ public final class GroupInviteLinkUrl {
         this.url = createUrl(groupMasterKey, password);
     }
 
-    protected static String createUrl(GroupMasterKey groupMasterKey, GroupLinkPassword password) {
+    private static String createUrl(GroupMasterKey groupMasterKey, GroupLinkPassword password) {
         var groupInviteLink = GroupInviteLink.newBuilder()
                 .setV1Contents(GroupInviteLink.GroupInviteLinkContentsV1.newBuilder()
                         .setGroupMasterKey(ByteString.copyFrom(groupMasterKey.serialize()))

lib/src/main/java/org/asamk/signal/manager/groups/GroupUtils.java

@@ -6,7 +6,7 @@ import org.asamk.signal.manager.storage.groups.GroupInfoV2;
 import org.signal.zkgroup.InvalidInputException;
 import org.signal.zkgroup.groups.GroupMasterKey;
 import org.signal.zkgroup.groups.GroupSecretParams;
-import org.whispersystems.libsignal.kdf.HKDFv3;
+import org.whispersystems.libsignal.kdf.HKDF;
 import org.whispersystems.signalservice.api.messages.SignalServiceDataMessage;
 import org.whispersystems.signalservice.api.messages.SignalServiceGroup;
 import org.whispersystems.signalservice.api.messages.SignalServiceGroupContext;
@@ -57,7 +57,7 @@ public class GroupUtils {
 
     private static GroupMasterKey deriveV2MigrationMasterKey(GroupIdV1 groupIdV1) {
         try {
-            return new GroupMasterKey(new HKDFv3().deriveSecrets(groupIdV1.serialize(),
+            return new GroupMasterKey(HKDF.deriveSecrets(groupIdV1.serialize(),
                     "GV2 Migration".getBytes(),
                     GroupMasterKey.SIZE));
         } catch (InvalidInputException e) {

lib/src/main/java/org/asamk/signal/manager/storage/SignalAccount.java

@@ -890,10 +890,6 @@ public class SignalAccount implements Closeable {
         save();
     }
 
-    public String getRegistrationLockPin() {
-        return registrationLockPin;
-    }
-
     public void setRegistrationLockPin(final String registrationLockPin, final MasterKey pinMasterKey) {
         this.registrationLockPin = registrationLockPin;
         this.pinMasterKey = pinMasterKey;

lib/src/main/java/org/asamk/signal/manager/storage/identities/IdentityInfo.java

@@ -41,8 +41,4 @@ public class IdentityInfo {
     public Date getDateAdded() {
         return this.added;
     }
-
-    public byte[] getFingerprint() {
-        return identityKey.getPublicKey().serialize();
-    }
 }

lib/src/main/java/org/asamk/signal/manager/util/KeyUtils.java

@@ -68,10 +68,6 @@ public class KeyUtils {
         return new SignedPreKeyRecord(signedPreKeyId, System.currentTimeMillis(), keyPair, signature);
     }
 
-    public static String createSignalingKey() {
-        return getSecret(52);
-    }
-
     public static ProfileKey createProfileKey() {
         try {
             return new ProfileKey(getSecretBytes(32));

src/main/java/org/asamk/signal/dbus/DbusManagerImpl.java

@@ -131,7 +131,7 @@ public class DbusManagerImpl implements Manager {
                 emptyIfNull(about),
                 emptyIfNull(aboutEmoji),
                 avatar == null ? "" : avatar.map(File::getPath).orElse(""),
-                avatar != null && !avatar.isPresent());
+                avatar != null && avatar.isEmpty());
     }
 
     @Override
@@ -786,9 +786,8 @@ public class DbusManagerImpl implements Manager {
 
     private void notifyMessageHandlers(final MessageEnvelope envelope) {
         synchronized (messageHandlers) {
-            Stream.concat(messageHandlers.stream(), weakHandlers.stream()).forEach(h -> {
+            Stream.concat(messageHandlers.stream(), weakHandlers.stream())
-                h.handleMessage(envelope, null);
+                    .forEach(h -> h.handleMessage(envelope, null));
-            });
         }
     }
 

src/main/java/org/asamk/signal/dbus/DbusReceiveMessageHandler.java

@@ -18,12 +18,10 @@ import java.util.stream.Collectors;
 
 public class DbusReceiveMessageHandler implements Manager.ReceiveMessageHandler {
 
-    private final Manager m;
     private final DBusConnection conn;
     private final String objectPath;
 
-    public DbusReceiveMessageHandler(Manager m, DBusConnection conn, final String objectPath) {
+    public DbusReceiveMessageHandler(DBusConnection conn, final String objectPath) {
-        this.m = m;
         this.conn = conn;
         this.objectPath = objectPath;
     }
@@ -127,7 +125,7 @@ public class DbusReceiveMessageHandler implements Manager.ReceiveMessageHandler
             var attachments = message.attachments()
                     .stream()
                     .filter(a -> a.id().isPresent())
-                    .map(a -> getAttachmentMap(m, a))
+                    .map(this::getAttachmentMap)
                     .collect(Collectors.toList());
             extras.put("attachments", new Variant<>(attachments, "aa{sv}"));
         }
@@ -180,7 +178,7 @@ public class DbusReceiveMessageHandler implements Manager.ReceiveMessageHandler
     }
 
     private Map<String, Variant<?>> getAttachmentMap(
-            final Manager m, final MessageEnvelope.Data.Attachment a
+            final MessageEnvelope.Data.Attachment a
     ) {
         final var map = new HashMap<String, Variant<?>>();
         if (a.id().isPresent()) {

src/main/java/org/asamk/signal/dbus/DbusSignalImpl.java

@@ -111,7 +111,7 @@ public class DbusSignalImpl implements Signal {
     @Override
     public void subscribeReceive() {
         if (dbusMessageHandler == null) {
-            dbusMessageHandler = new DbusReceiveMessageHandler(m, connection, objectPath);
+            dbusMessageHandler = new DbusReceiveMessageHandler(connection, objectPath);
             m.addReceiveHandler(dbusMessageHandler);
         }
         subscriberCount++;

src/main/java/org/asamk/signal/json/JsonAttachment.java

@@ -10,8 +10,4 @@ record JsonAttachment(String contentType, String filename, String id, Long size)
         final var size = attachment.size().orElse(null);
         return new JsonAttachment(attachment.contentType(), filename, id, size);
     }
-
-    static JsonAttachment from(String filename) {
-        return new JsonAttachment(filename, null, null, null);
-    }
 }

src/main/java/org/asamk/signal/json/JsonDataMessage.java

@@ -2,7 +2,6 @@ package org.asamk.signal.json;
 
 import com.fasterxml.jackson.annotation.JsonInclude;
 
-import org.asamk.Signal;
 import org.asamk.signal.manager.api.MessageEnvelope;
 
 import java.util.List;
@@ -65,38 +64,4 @@ record JsonDataMessage(
                 contacts,
                 groupInfo);
     }
-
-    static JsonDataMessage from(Signal.MessageReceived messageReceived) {
-        return new JsonDataMessage(messageReceived.getTimestamp(),
-                messageReceived.getMessage(),
-                // TODO Replace these with the proper commands
-                null,
-                null,
-                null,
-                null,
-                null,
-                null,
-                messageReceived.getAttachments().stream().map(JsonAttachment::from).collect(Collectors.toList()),
-                null,
-                null,
-                null,
-                messageReceived.getGroupId().length > 0 ? JsonGroupInfo.from(messageReceived.getGroupId()) : null);
-    }
-
-    static JsonDataMessage from(Signal.SyncMessageReceived messageReceived) {
-        return new JsonDataMessage(messageReceived.getTimestamp(),
-                messageReceived.getMessage(),
-                // TODO Replace these with the proper commands
-                null,
-                null,
-                null,
-                null,
-                null,
-                null,
-                messageReceived.getAttachments().stream().map(JsonAttachment::from).collect(Collectors.toList()),
-                null,
-                null,
-                null,
-                messageReceived.getGroupId().length > 0 ? JsonGroupInfo.from(messageReceived.getGroupId()) : null);
-    }
 }

src/main/java/org/asamk/signal/json/JsonGroupInfo.java

@@ -2,16 +2,10 @@ package org.asamk.signal.json;
 
 import org.asamk.signal.manager.api.MessageEnvelope;
 
-import java.util.Base64;
-
 record JsonGroupInfo(String groupId, String type) {
 
     static JsonGroupInfo from(MessageEnvelope.Data.GroupContext groupContext) {
         return new JsonGroupInfo(groupContext.groupId().toBase64(),
                 groupContext.isGroupUpdate() ? "UPDATE" : "DELIVER");
     }
-
-    static JsonGroupInfo from(byte[] groupId) {
-        return new JsonGroupInfo(Base64.getEncoder().encodeToString(groupId), "DELIVER");
-    }
 }

src/main/java/org/asamk/signal/util/ErrorUtils.java

@@ -5,8 +5,6 @@ import org.asamk.signal.commands.exceptions.IOErrorException;
 import org.asamk.signal.manager.api.ProofRequiredException;
 import org.asamk.signal.manager.api.RecipientIdentifier;
 import org.asamk.signal.manager.api.SendMessageResult;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 import java.util.ArrayList;
 import java.util.Collection;
@@ -16,8 +14,6 @@ import java.util.stream.Collectors;
 
 public class ErrorUtils {
 
-    private final static Logger logger = LoggerFactory.getLogger(ErrorUtils.class);
-
     private ErrorUtils() {
     }