X-Git-Url: https://git.nmode.ca/signal-cli/blobdiff_plain/de273586b4106171a3940ce5ec2c2b3712430c8c..48b2c7ff8ad7b653c3a0cb42775a5850f593d0a0:/src/main/java/org/asamk/signal/commands/TrustCommand.java diff --git a/src/main/java/org/asamk/signal/commands/TrustCommand.java b/src/main/java/org/asamk/signal/commands/TrustCommand.java index 277291eb..eb99386b 100644 --- a/src/main/java/org/asamk/signal/commands/TrustCommand.java +++ b/src/main/java/org/asamk/signal/commands/TrustCommand.java @@ -4,8 +4,10 @@ import net.sourceforge.argparse4j.impl.Arguments; import net.sourceforge.argparse4j.inf.Namespace; import net.sourceforge.argparse4j.inf.Subparser; +import org.asamk.signal.OutputWriter; +import org.asamk.signal.commands.exceptions.CommandException; +import org.asamk.signal.commands.exceptions.UserErrorException; import org.asamk.signal.manager.Manager; -import org.asamk.signal.util.ErrorUtils; import org.asamk.signal.util.Hex; import org.whispersystems.signalservice.api.util.InvalidNumberException; @@ -13,8 +15,11 @@ import java.util.Locale; public class TrustCommand implements LocalCommand { - @Override - public void attachToSubparser(final Subparser subparser) { + public TrustCommand(final OutputWriter outputWriter) { + } + + public static void attachToSubparser(final Subparser subparser) { + subparser.help("Set the trust level of a given number."); subparser.addArgument("number").help("Specify the phone number, for which to set the trust.").required(true); var mutTrust = subparser.addMutuallyExclusiveGroup(); mutTrust.addArgument("-a", "--trust-all-known-keys") @@ -25,16 +30,20 @@ public class TrustCommand implements LocalCommand { } @Override - public int handleCommand(final Namespace ns, final Manager m) { + public void handleCommand(final Namespace ns, final Manager m) throws CommandException { var number = ns.getString("number"); - if (ns.getBoolean("trust_all_known_keys")) { - var res = m.trustIdentityAllKeys(number); + if (ns.getBoolean("trust-all-known-keys")) { + boolean res; + try { + res = m.trustIdentityAllKeys(number); + } catch (InvalidNumberException e) { + throw new UserErrorException("Failed to parse recipient: " + e.getMessage()); + } if (!res) { - System.err.println("Failed to set the trust for this number, make sure the number is correct."); - return 1; + throw new UserErrorException("Failed to set the trust for this number, make sure the number is correct."); } } else { - var safetyNumber = ns.getString("verified_safety_number"); + var safetyNumber = ns.getString("verified-safety-number"); if (safetyNumber != null) { safetyNumber = safetyNumber.replaceAll(" ", ""); if (safetyNumber.length() == 66) { @@ -42,46 +51,38 @@ public class TrustCommand implements LocalCommand { try { fingerprintBytes = Hex.toByteArray(safetyNumber.toLowerCase(Locale.ROOT)); } catch (Exception e) { - System.err.println( + throw new UserErrorException( "Failed to parse the fingerprint, make sure the fingerprint is a correctly encoded hex string without additional characters."); - return 1; } boolean res; try { res = m.trustIdentityVerified(number, fingerprintBytes); } catch (InvalidNumberException e) { - ErrorUtils.handleInvalidNumberException(e); - return 1; + throw new UserErrorException("Failed to parse recipient: " + e.getMessage()); } if (!res) { - System.err.println( + throw new UserErrorException( "Failed to set the trust for the fingerprint of this number, make sure the number and the fingerprint are correct."); - return 1; } } else if (safetyNumber.length() == 60) { boolean res; try { res = m.trustIdentityVerifiedSafetyNumber(number, safetyNumber); } catch (InvalidNumberException e) { - ErrorUtils.handleInvalidNumberException(e); - return 1; + throw new UserErrorException("Failed to parse recipient: " + e.getMessage()); } if (!res) { - System.err.println( + throw new UserErrorException( "Failed to set the trust for the safety number of this phone number, make sure the phone number and the safety number are correct."); - return 1; } } else { - System.err.println( + throw new UserErrorException( "Safety number has invalid format, either specify the old hex fingerprint or the new safety number"); - return 1; } } else { - System.err.println( + throw new UserErrorException( "You need to specify the fingerprint/safety number you have verified with -v SAFETY_NUMBER"); - return 1; } } - return 0; } }