Merge pull request #123 from NASA-PDS/issue_121

Improve `list-dd` CLI command argument handling
NASA-PDS · Dec 12, 2024 · 4692d60 · 4692d60
2 parents a6fdae4 + 2c2101a
commit 4692d60
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/src/main/java/gov/nasa/pds/registry/mgr/cmd/dd/ListDDCmd.java b/src/main/java/gov/nasa/pds/registry/mgr/cmd/dd/ListDDCmd.java
@@ -34,11 +34,16 @@ public void run(CommandLine cmdLine) throws Exception
             return;
         }
 
-        String url = cmdLine.getOptionValue("es", "app:/connections/direct/localhost.xml");
         String authFile = cmdLine.getOptionValue("auth");
-
         String namespace = cmdLine.getOptionValue("ns");
-
+        String url = cmdLine.getOptionValue("es", "app:/connections/direct/localhost.xml");
+
+        if (authFile == null || authFile.isBlank()) {
+          throw new IllegalArgumentException("missing argument: -auth must be given");
+        }
+        if (namespace == null || namespace.isBlank()) {
+          throw new IllegalArgumentException("missing argument: -ns must be given");         
+        }
         try
         {
             RegistryManager.init(url, authFile);