Correctly parse and pass on arguments to serve

Fixes mistake in b3f4355

Author: kevmoo

webdev/lib/src/command/command_base.dart

@@ -45,13 +45,8 @@ abstract class CommandBase extends Command<int> {
           help: 'Enables verbose logging.');
   }
 
-  Future<int> runCore(String command) async {
-    await checkPubspecLock();
-
-    var buildRunnerScript = await _buildRunnerScript();
-
-    final arguments = [command];
-
+  List<String> getArgs() {
+    var arguments = <String>[];
     if ((argResults[_release] as bool) ?? releaseDefault) {
       arguments.add('--$_release');
     }
@@ -64,6 +59,15 @@ abstract class CommandBase extends Command<int> {
     if (argResults[_verbose] as bool) {
       arguments.add('--$_verbose');
     }
+    return arguments;
+  }
+
+  Future<int> runCore(String command) async {
+    await checkPubspecLock();
+
+    var buildRunnerScript = await _buildRunnerScript();
+
+    final arguments = [command]..addAll(getArgs());
 
     var exitCode = 0;
 

webdev/lib/src/command/serve_command.dart

@@ -30,6 +30,22 @@ class ServeCommand extends CommandBase {
           help: 'Enables logging for each request to the server.');
   }
 
+  @override
+  List<String> getArgs() {
+    var arguments = super.getArgs();
+
+    var hostname = argResults['hostname'] as String;
+    if (hostname != null) {
+      arguments.addAll(['--hostname', hostname]);
+    }
+
+    if (argResults['log-requests'] == true) {
+      arguments.add('--log-requests');
+    }
+
+    return arguments;
+  }
+
   @override
   Future<int> run() => runCore('serve');
 }