Fix loglevel options (#323)

* Fix new log levels * Update readme
Azure-Samples · Nov 28, 2023 · 4af23f7 · 4af23f7
1 parent 170c862
commit 4af23f7
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 8 deletions.
diff --git a/README.md b/README.md
@@ -292,8 +292,8 @@ Options:
                              the timespan in seconds when the logfile should be
                                flushed.
                                Default: 00:00:30 sec
-      --ll, --loglevel=VALUE the loglevel to use (allowed: fatal, error, warn,
-                               info, debug, verbose).
+      --ll, --loglevel=VALUE the loglevel to use (allowed: critical, error,
+                               warn, info, debug, trace).
                                Default: info
       --sc, --simulationcyclecount=VALUE
                              count of cycles in one simulation phase
@@ -337,14 +337,15 @@ Options:
       --to, --trustowncert   the own certificate is put into the trusted
                                certificate store automatically.
                                Default: False
-      --msec, --maxsessioncount
+      --msec, --maxsessioncount=VALUE
                              maximum number of parallel sessions.
                                Default: 100
-      --msuc, --maxsubscriptioncount
+      --msuc, --maxsubscriptioncount=VALUE
                              maximum number of subscriptions.
                                Default: 100
-      --mqrc, --maxqueuedrequestcount
-                             maximum number of requests that will be queued waiting for a thread.
+      --mqrc, --maxqueuedrequestcount=VALUE
+                             maximum number of requests that will be queued
+                               waiting for a thread.
                                Default: 2000
       --at, --appcertstoretype=VALUE
                              the own application cert store type.

diff --git a/src/CliOptions.cs b/src/CliOptions.cs
@@ -32,15 +32,15 @@ public static Mono.Options.OptionSet InitCommandLineOptions()
                     }
                 }
             },
-            { "ll|loglevel=", "the loglevel to use (allowed: critical, warn, info, debug, trace).\nDefault: info", (string s) => {
+            { "ll|loglevel=", "the loglevel to use (allowed: critical, error, warn, info, debug, trace).\nDefault: info", (string s) => {
                     var logLevels = new List<string> {"critical", "error", "warn", "info", "debug", "trace"};
                     if (logLevels.Contains(s.ToLowerInvariant()))
                     {
                         Program.LogLevelCli = s.ToLowerInvariant();
                     }
                     else
                     {
-                        throw new OptionException("The loglevel must be one of: fatal, error, warn, info, debug, verbose", "loglevel");
+                        throw new OptionException($"The loglevel must be one of: {string.Join(", ", logLevels)}", "loglevel");
                     }
                 }
             },