Fix for urls command line parameter (#44)

ttu · Feb 7, 2019 · ece98b3 · ece98b3
1 parent ee2e79d
commit ece98b3
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,7 @@
 # Changelog
 
 ### [Unreleased]
+* FIXED: urls command line parameter is not ignored
 
 ### [0.9.0] - 2019-01-31
 * ADDED: Support for paging with page and per_page

diff --git a/FakeServer/Program.cs b/FakeServer/Program.cs
@@ -71,7 +71,7 @@ private static CommandLineApplication BuildCommandLineApp(
                 CommandOptionType.SingleValue);
             var optionServe = app.Option("-s|--serve <PATH>", "Static files (default wwwroot)",
                 CommandOptionType.SingleValue);
-            app.Option("--urls <URLS>", "Server url (default http://localhost:57602)", CommandOptionType.SingleValue);
+            var optionsUrls = app.Option("--urls <URLS>", "Server url (default http://localhost:57602)", CommandOptionType.SingleValue);
 
             app.OnExecute(() =>
             {
@@ -107,6 +107,13 @@ private static CommandLineApplication BuildCommandLineApp(
                     Console.WriteLine($"Static files: default wwwroot");
                 }
 
+                if (optionsUrls.HasValue())
+                {
+                    // Add urls back to arguments that are passed to WebHost builder
+                    app.RemainingArguments.Add("--urls");
+                    app.RemainingArguments.Add(optionsUrls.Value());
+                }
+
                 return invoke(app.RemainingArguments.ToArray(), initialData);
             });
 
@@ -118,4 +125,4 @@ private static string GetAssemblyVersion()
             return FileVersionInfo.GetVersionInfo(Assembly.GetExecutingAssembly().Location).ProductVersion;
         }
     }
-}
+}