Merged
Conversation
4ba4567 to
bf7815f
Compare
Member
|
The nginx redirects being added by |
bf7815f to
4f1b4d5
Compare
Member
Author
|
Good catch — fixed. The |
f7f5dbb to
16687e8
Compare
maartenba
requested changes
Feb 26, 2026
Member
maartenba
left a comment
There was a problem hiding this comment.
Use writing guidelines/styles
16687e8 to
c8ceb10
Compare
c8ceb10 to
c0bbc9c
Compare
|
Preview Environment Provisioned |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Replace the nginx-based static file serving with an ASP.NET Core application and restructure the repository to separate Astro content from the .NET server.
What changed
Architecture: nginx → ASP.NET Core (
server/src/Docs.Web)redirects.json(generated by Astro build fromredirect_fromfrontmatter) and performs 301 redirects with case-insensitive matching.dotnet publish /t:PublishContainertargetingaspnet:10.0-alpine.Dockerfile,.dockerignore, anddeployment/(nginx configs).Repository restructure
astro/subdirectory to cleanly separate content from the .NET server.server/src/andserver/tests/subdirectories.node:22-slim) to avoid cross-platform npm issues with native modules likesharpandresvg-js.Build orchestration (
build.cs)astro-build,dotnet-build,dotnet-test,container,aspire,link-check,verify-formatting,clean.dotnet build.cs <target>.Integration tests (
server/tests/Docs.Web.Tests)WebApplicationFactorywith isolated tempwwwrootand testredirects.json.Local development with .NET Aspire (
server/src/Docs.AppHost)dotnet build.cs aspireorchestrates the Astro dev server (port 4321) and the ASP.NET Core app together with OpenTelemetry, health checks, and the Aspire dashboard.New project structure
Build & run
Files removed
Dockerfile,.dockerignoredeployment/default.conf,deployment/nginx.confbuild.sh