diff options
author | Matthew Harmsen <mharmsen@redhat.com> | 2014-04-16 18:38:10 -0700 |
---|---|---|
committer | Matthew Harmsen <mharmsen@redhat.com> | 2014-04-17 11:24:50 -0700 |
commit | 611419fcd8a19c06ca651add93deb66bdd0c55d5 (patch) | |
tree | 4c2ca4e4f2253556aa08f38f27e215f78a1c1383 /base/java-tools/src/com/netscape/cmstools/tps/token/TokenAddCLI.java | |
parent | 7c7cbb84f1c43ce3fbe33676f3e0f07fb7ecc2f9 (diff) | |
download | pki-611419fcd8a19c06ca651add93deb66bdd0c55d5.tar.gz pki-611419fcd8a19c06ca651add93deb66bdd0c55d5.tar.xz pki-611419fcd8a19c06ca651add93deb66bdd0c55d5.zip |
CLI argument parsing and bad return codes
* PKI TRAC Ticket #843 - Incorrect CLI argument parsing
* PKI TRAC Ticket #918 - CLI commands does not return code '1' for the failures
Diffstat (limited to 'base/java-tools/src/com/netscape/cmstools/tps/token/TokenAddCLI.java')
-rw-r--r-- | base/java-tools/src/com/netscape/cmstools/tps/token/TokenAddCLI.java | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/base/java-tools/src/com/netscape/cmstools/tps/token/TokenAddCLI.java b/base/java-tools/src/com/netscape/cmstools/tps/token/TokenAddCLI.java index bb9d452a5..396dbc63f 100644 --- a/base/java-tools/src/com/netscape/cmstools/tps/token/TokenAddCLI.java +++ b/base/java-tools/src/com/netscape/cmstools/tps/token/TokenAddCLI.java @@ -18,6 +18,8 @@ package com.netscape.cmstools.tps.token; +import java.util.Arrays; + import org.apache.commons.cli.CommandLine; import org.apache.commons.cli.Option; @@ -43,6 +45,13 @@ public class TokenAddCLI extends CLI { public void execute(String[] args) throws Exception { + // Check for "--help" prior to parsing due to required option + if (Arrays.asList(args).contains("--help")) { + // Display usage + printHelp(); + System.exit(0); + } + Option option = new Option(null, "user", true, "User ID"); option.setArgName("User ID"); option.setRequired(true); |