BREAKING CHANGE: Change default serve port to 3000 and add ability to override

README.md

@@ -25,7 +25,7 @@ Options:
 
 Commands:
   generate    Converts a source markdown folder or file to an output folder
-  serve       Continuously serve a documentation folder at http://localhost:5000.
+  serve       Continuously serve a documentation folder at http://localhost:3000.
     File systems changes will be reflected without having to restart the server.
 ```
 

docs/source/contribute/locally.md

@@ -29,7 +29,7 @@ Follow these steps to contribute to Elastic docs.
    ```
 
 3. **Run the Binary:**
-   Use the `serve` command to start serving the documentation at http://localhost:5000. The path to the docset.yml file that you want to build can be specified with `-p`:
+   Use the `serve` command to start serving the documentation at http://localhost:3000. The path to the docset.yml file that you want to build can be specified with `-p`:
    ```sh
    ./docs-builder serve -p ./path/to/docs
    ```
@@ -53,7 +53,7 @@ Follow these steps to contribute to Elastic docs.
    ```
 
 3. **Run the Binary:**
-   Use the `serve` command to start serving the documentation at http://localhost:5000. The path to the docset.yml file that you want to build can be specified with `-p`:
+   Use the `serve` command to start serving the documentation at http://localhost:3000. The path to the docset.yml file that you want to build can be specified with `-p`:
    ```sh
    .\docs-builder serve -p ./path/to/docs
    ```
@@ -77,7 +77,7 @@ Follow these steps to contribute to Elastic docs.
    ```
 
 3. **Run the Binary:**
-   Use the `serve` command to start serving the documentation at http://localhost:5000. The path to the docset.yml file that you want to build can be specified with `-p`:
+   Use the `serve` command to start serving the documentation at http://localhost:3000. The path to the docset.yml file that you want to build can be specified with `-p`:
    ```sh
    ./docs-builder serve -p ./path/to/docs
    ```
@@ -101,7 +101,7 @@ git clone https://github.com/elastic/docs-content.git
    ```
 
 2. **Run the Binary:**
-   Use the `serve` command to start serving the documentation at http://localhost:5000. The path to the `docset.yml` file that you want to build can be specified with `-p`:
+   Use the `serve` command to start serving the documentation at http://localhost:3000. The path to the `docset.yml` file that you want to build can be specified with `-p`:
    ```sh
    # macOS/Linux
    ./docs-builder serve -p ./migration-test
@@ -110,7 +110,7 @@ git clone https://github.com/elastic/docs-content.git
    .\docs-builder serve -p .\migration-test
    ```
 
-Now you should be able to view the documentation locally by navigating to http://localhost:5000.
+Now you should be able to view the documentation locally by navigating to http://localhost:3000.
 
 ## Step 4: Open a PR [#step-four]
 

src/docs-builder/Cli/Commands.cs

@@ -16,20 +16,20 @@ namespace Documentation.Builder.Cli;
 internal class Commands(ILoggerFactory logger, ICoreService githubActionsService)
 {
 	/// <summary>
-	///	Continuously serve a documentation folder at http://localhost:5000.
+	///	Continuously serve a documentation folder at http://localhost:3000.
 	/// File systems changes will be reflected without having to restart the server.
 	/// </summary>
 	/// <param name="path">-p, Path to serve the documentation.
 	/// Defaults to the`{pwd}/docs` folder
 	/// </param>
+	/// <param name="port">Port to serve the documentation.</param>
 	/// <param name="ctx"></param>
 	[Command("serve")]
-	public async Task Serve(string? path = null, Cancel ctx = default)
+	public async Task Serve(string? path = null, int port = 3000, Cancel ctx = default)
 	{
-		var host = new DocumentationWebHost(path, logger, new FileSystem());
+		var host = new DocumentationWebHost(path, port, logger, new FileSystem());
 		await host.RunAsync(ctx);
 		await host.StopAsync(ctx);
-
 	}
 
 	/// <summary>

src/docs-builder/Http/DocumentationWebHost.cs

@@ -10,6 +10,7 @@
 using Elastic.Markdown.Diagnostics;
 using Elastic.Markdown.IO;
 using Microsoft.AspNetCore.Builder;
+using Microsoft.AspNetCore.Hosting;
 using Microsoft.AspNetCore.Http;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.FileProviders;
@@ -28,7 +29,7 @@ public class DocumentationWebHost
 
 	private readonly BuildContext _context;
 
-	public DocumentationWebHost(string? path, ILoggerFactory logger, IFileSystem fileSystem)
+	public DocumentationWebHost(string? path, int port, ILoggerFactory logger, IFileSystem fileSystem)
 	{
 		var builder = WebApplication.CreateSlimBuilder();
 
@@ -54,6 +55,12 @@ public DocumentationWebHost(string? path, ILoggerFactory logger, IFileSystem fil
 
 		//builder.Services.AddSingleton(logger);
 
+		// See https://learn.microsoft.com/en-us/dotnet/core/tools/dotnet-environment-variables#dotnet_running_in_container-and-dotnet_running_in_containers
+		// This way use the default port when running in a container
+		if (string.IsNullOrEmpty(Environment.GetEnvironmentVariable("DOTNET_RUNNING_IN_CONTAINER")) &&
+		    string.IsNullOrEmpty(Environment.GetEnvironmentVariable("DOTNET_RUNNING_IN_CONTAINERS")))
+			builder.WebHost.UseUrls($"http://localhost:{port}");
+
 		_webApplication = builder.Build();
 		SetUpRoutes();
 	}