]> nmode's Git Repositories - signal-cli/commitdiff
Remove error handling for oracle unlimited crypto issue
authorAsamK <asamk@gmx.de>
Sat, 1 May 2021 06:45:10 +0000 (08:45 +0200)
committerAsamK <asamk@gmx.de>
Sat, 1 May 2021 06:45:10 +0000 (08:45 +0200)
README.md
src/main/java/org/asamk/signal/commands/AddDeviceCommand.java
src/main/java/org/asamk/signal/commands/JoinGroupCommand.java
src/main/java/org/asamk/signal/commands/LinkCommand.java
src/main/java/org/asamk/signal/commands/QuitGroupCommand.java
src/main/java/org/asamk/signal/commands/ReceiveCommand.java
src/main/java/org/asamk/signal/commands/RemoteDeleteCommand.java
src/main/java/org/asamk/signal/commands/SendCommand.java
src/main/java/org/asamk/signal/commands/SendReactionCommand.java
src/main/java/org/asamk/signal/commands/UpdateGroupCommand.java
src/main/java/org/asamk/signal/util/ErrorUtils.java

index 5eb9e78d685af23c61c4574c3f859edbc143b6bd..9d60499c2a69491b2c11d41aaf0364b3cc01bed5 100644 (file)
--- a/README.md
+++ b/README.md
@@ -102,8 +102,8 @@ This is still experimental and will not work in all situations.
   
    The binary is available at *build/native-image/signal-cli*
 
-## Troubleshooting
-If you use a version of the Oracle JRE and get an InvalidKeyException you need to enable unlimited strength crypto. See https://stackoverflow.com/questions/6481627/java-security-illegal-key-size-or-default-parameters for instructions.
+## FAQ and Troubleshooting
+For frequently asked questions and issues have a look at the [wiki](https://github.com/AsamK/signal-cli/wiki/FAQ)
 
 ## License
 
index cf993e6de4b89d7064d790df40210043374200f1..7a66babfe8a0ba06603c4c4f0769a258ba731bcc 100644 (file)
@@ -16,8 +16,6 @@ import java.io.IOException;
 import java.net.URI;
 import java.net.URISyntaxException;
 
-import static org.asamk.signal.util.ErrorUtils.handleAssertionError;
-
 public class AddDeviceCommand implements LocalCommand {
 
     private final static Logger logger = LoggerFactory.getLogger(AddDeviceCommand.class);
@@ -41,9 +39,6 @@ public class AddDeviceCommand implements LocalCommand {
         } catch (InvalidKeyException e) {
             logger.error("Add device link failed", e);
             throw new UnexpectedErrorException("Add device link failed.");
-        } catch (AssertionError e) {
-            handleAssertionError(e);
-            throw e;
         }
     }
 }
index e59ecec644e6262f589bf68ce97c9177da5cfcd8..bcd88ab415265b849a02408e76e201599e097d72 100644 (file)
@@ -16,7 +16,6 @@ import org.whispersystems.signalservice.internal.push.exceptions.GroupPatchNotAc
 
 import java.io.IOException;
 
-import static org.asamk.signal.util.ErrorUtils.handleAssertionError;
 import static org.asamk.signal.util.ErrorUtils.handleTimestampAndSendMessageResults;
 
 public class JoinGroupCommand implements LocalCommand {
@@ -53,9 +52,6 @@ public class JoinGroupCommand implements LocalCommand {
                 writer.println("Joined group \"{}\"", newGroupId.toBase64());
             }
             handleTimestampAndSendMessageResults(writer, 0, results.second());
-        } catch (AssertionError e) {
-            handleAssertionError(e);
-            throw e;
         } catch (GroupPatchNotAcceptedException e) {
             throw new UserErrorException("Failed to join group, maybe already a member");
         } catch (IOException e) {
index 3a0186195995f14f1d300b5c007972dade4fb890..cd2e188e35f3fdf792274cec4efe23cdd03258ac 100644 (file)
@@ -17,8 +17,6 @@ import org.whispersystems.libsignal.InvalidKeyException;
 import java.io.IOException;
 import java.util.concurrent.TimeoutException;
 
-import static org.asamk.signal.util.ErrorUtils.handleAssertionError;
-
 public class LinkCommand implements ProvisioningCommand {
 
     private final static Logger logger = LoggerFactory.getLogger(LinkCommand.class);
@@ -45,9 +43,6 @@ public class LinkCommand implements ProvisioningCommand {
             throw new UserErrorException("Link request timed out, please try again.");
         } catch (IOException e) {
             throw new IOErrorException("Link request error: " + e.getMessage());
-        } catch (AssertionError e) {
-            handleAssertionError(e);
-            throw e;
         } catch (InvalidKeyException e) {
             logger.debug("Finish device link failed", e);
             throw new UnexpectedErrorException("Invalid key: " + e.getMessage());
index d8a865854e3110b3cd3f3d39d3a2ca8741a55dd8..e47728d5278f49e69c3778e4795c48dc1112d4ee 100644 (file)
@@ -16,7 +16,6 @@ import org.asamk.signal.util.Util;
 
 import java.io.IOException;
 
-import static org.asamk.signal.util.ErrorUtils.handleAssertionError;
 import static org.asamk.signal.util.ErrorUtils.handleTimestampAndSendMessageResults;
 
 public class QuitGroupCommand implements LocalCommand {
@@ -42,12 +41,7 @@ public class QuitGroupCommand implements LocalCommand {
             handleTimestampAndSendMessageResults(writer, results.first(), results.second());
         } catch (IOException e) {
             throw new IOErrorException("Failed to send message: " + e.getMessage());
-        } catch (AssertionError e) {
-            handleAssertionError(e);
-            throw e;
-        } catch (GroupNotFoundException e) {
-            throw new UserErrorException("Failed to send to group: " + e.getMessage());
-        } catch (NotAGroupMemberException e) {
+        } catch (GroupNotFoundException | NotAGroupMemberException e) {
             throw new UserErrorException("Failed to send to group: " + e.getMessage());
         }
     }
index 8612a71b631d27a6d1632ef07b6cde58857e736e..a03fc130d3889505a3769d74664f09f017825b9c 100644 (file)
@@ -27,8 +27,6 @@ import java.util.Map;
 import java.util.Set;
 import java.util.concurrent.TimeUnit;
 
-import static org.asamk.signal.util.ErrorUtils.handleAssertionError;
-
 public class ReceiveCommand implements ExtendedDbusCommand, LocalCommand {
 
     private final static Logger logger = LoggerFactory.getLogger(ReceiveCommand.class);
@@ -169,9 +167,6 @@ public class ReceiveCommand implements ExtendedDbusCommand, LocalCommand {
                     handler);
         } catch (IOException e) {
             throw new IOErrorException("Error while receiving messages: " + e.getMessage());
-        } catch (AssertionError e) {
-            handleAssertionError(e);
-            throw e;
         }
     }
 }
index 796a3344d9007e793d2e93b4fd434ad0d64e28be..793b362a9521e7a13888307ae681bb1dd4d8019b 100644 (file)
@@ -15,8 +15,6 @@ import org.freedesktop.dbus.exceptions.DBusExecutionException;
 
 import java.util.List;
 
-import static org.asamk.signal.util.ErrorUtils.handleAssertionError;
-
 public class RemoteDeleteCommand implements DbusCommand {
 
     @Override
@@ -26,10 +24,8 @@ public class RemoteDeleteCommand implements DbusCommand {
                 .required(true)
                 .type(long.class)
                 .help("Specify the timestamp of the message to delete.");
-        subparser.addArgument("-g", "--group")
-                .help("Specify the recipient group ID.");
-        subparser.addArgument("recipient")
-                .help("Specify the recipients' phone number.").nargs("*");
+        subparser.addArgument("-g", "--group").help("Specify the recipient group ID.");
+        subparser.addArgument("recipient").help("Specify the recipients' phone number.").nargs("*");
     }
 
     @Override
@@ -66,9 +62,6 @@ public class RemoteDeleteCommand implements DbusCommand {
                 timestamp = signal.sendRemoteDeleteMessage(targetTimestamp, recipients);
             }
             writer.println("{}", timestamp);
-        } catch (AssertionError e) {
-            handleAssertionError(e);
-            throw e;
         } catch (UnknownObject e) {
             throw new UserErrorException("Failed to find dbus object, maybe missing the -u flag: " + e.getMessage());
         } catch (Signal.Error.InvalidNumber e) {
index bd94ca91b6de4e1fb597b6a4b7e7a5f5fcb329c1..71de11308c307da938c7850990f6f3b6a14ad8fd 100644 (file)
@@ -22,8 +22,6 @@ import java.io.IOException;
 import java.nio.charset.Charset;
 import java.util.List;
 
-import static org.asamk.signal.util.ErrorUtils.handleAssertionError;
-
 public class SendCommand implements DbusCommand {
 
     private final static Logger logger = LoggerFactory.getLogger(SendCommand.class);
@@ -67,9 +65,6 @@ public class SendCommand implements DbusCommand {
             try {
                 signal.sendEndSessionMessage(recipients);
                 return;
-            } catch (AssertionError e) {
-                handleAssertionError(e);
-                throw e;
             } catch (Signal.Error.UntrustedIdentity e) {
                 throw new UntrustedKeyErrorException("Failed to send message: " + e.getMessage());
             } catch (DBusExecutionException e) {
@@ -105,9 +100,6 @@ public class SendCommand implements DbusCommand {
                 var timestamp = signal.sendGroupMessage(messageText, attachments, groupId);
                 writer.println("{}", timestamp);
                 return;
-            } catch (AssertionError e) {
-                handleAssertionError(e);
-                throw e;
             } catch (DBusExecutionException e) {
                 throw new UnexpectedErrorException("Failed to send group message: " + e.getMessage());
             }
@@ -118,9 +110,6 @@ public class SendCommand implements DbusCommand {
                 var timestamp = signal.sendNoteToSelfMessage(messageText, attachments);
                 writer.println("{}", timestamp);
                 return;
-            } catch (AssertionError e) {
-                handleAssertionError(e);
-                throw e;
             } catch (Signal.Error.UntrustedIdentity e) {
                 throw new UntrustedKeyErrorException("Failed to send message: " + e.getMessage());
             } catch (DBusExecutionException e) {
@@ -131,9 +120,6 @@ public class SendCommand implements DbusCommand {
         try {
             var timestamp = signal.sendMessage(messageText, attachments, recipients);
             writer.println("{}", timestamp);
-        } catch (AssertionError e) {
-            handleAssertionError(e);
-            throw e;
         } catch (UnknownObject e) {
             throw new UserErrorException("Failed to find dbus object, maybe missing the -u flag: " + e.getMessage());
         } catch (Signal.Error.UntrustedIdentity e) {
index a80b6a23bc72fe1273eabca6f82b83c153242361..f610e81b973098f252f0454b2fc36842bf01d2ed 100644 (file)
@@ -16,8 +16,6 @@ import org.freedesktop.dbus.exceptions.DBusExecutionException;
 
 import java.util.List;
 
-import static org.asamk.signal.util.ErrorUtils.handleAssertionError;
-
 public class SendReactionCommand implements DbusCommand {
 
     @Override
@@ -75,9 +73,6 @@ public class SendReactionCommand implements DbusCommand {
                 timestamp = signal.sendMessageReaction(emoji, isRemove, targetAuthor, targetTimestamp, recipients);
             }
             writer.println("{}", timestamp);
-        } catch (AssertionError e) {
-            handleAssertionError(e);
-            throw e;
         } catch (UnknownObject e) {
             throw new UserErrorException("Failed to find dbus object, maybe missing the -u flag: " + e.getMessage());
         } catch (Signal.Error.InvalidNumber e) {
index 204dcfe2af812e5afe06e0f3ba5fa94d018c7916..00730c82d117c3f0a22b337c52da15148bb74ddf 100644 (file)
@@ -18,8 +18,6 @@ import java.util.ArrayList;
 import java.util.Base64;
 import java.util.List;
 
-import static org.asamk.signal.util.ErrorUtils.handleAssertionError;
-
 public class UpdateGroupCommand implements DbusCommand {
 
     private final static Logger logger = LoggerFactory.getLogger(UpdateGroupCommand.class);
@@ -67,9 +65,6 @@ public class UpdateGroupCommand implements DbusCommand {
             if (groupId.length != newGroupId.length) {
                 writer.println("Created new group: \"{}\"", Base64.getEncoder().encodeToString(newGroupId));
             }
-        } catch (AssertionError e) {
-            handleAssertionError(e);
-            throw e;
         } catch (Signal.Error.AttachmentInvalid e) {
             throw new UserErrorException("Failed to add avatar attachment for group\": " + e.getMessage());
         } catch (DBusExecutionException e) {
index 595509a6fd62827befc8753ccc8ddf1ab6180497..100368f6dfa014ee1911eb8382b1ca8738f80157 100644 (file)
@@ -17,10 +17,6 @@ public class ErrorUtils {
     private ErrorUtils() {
     }
 
-    public static void handleAssertionError(AssertionError e) {
-        logger.warn("If you use an Oracle JRE please check if you have unlimited strength crypto enabled, see README");
-    }
-
     public static void handleTimestampAndSendMessageResults(
             PlainTextWriter writer, long timestamp, List<SendMessageResult> results
     ) throws CommandException {