Skip to content

Fix frontend build output path (Ingress → AuthProxy) - #58

Merged
woksin merged 1 commit into
mainfrom
fix/frontend-build-output-path
Jul 14, 2026
Merged

Fix frontend build output path (Ingress → AuthProxy)#58
woksin merged 1 commit into
mainfrom
fix/frontend-build-output-path

Conversation

@woksin

@woksin woksin commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Summary

The Vite build still pointed at ../Ingress/wwwroot — a project that no longer exists after the rename to AuthProxy — so npm run build never populated Source/AuthProxy/wwwroot. The frontend was absent from the published container image, and the publish pipeline's frontend artifact was silently empty (upload-artifact defaults to if-no-files-found: warn), which then broke the dotnet-publish step that downloads it.

Fixed

  • The React frontend is now built into the AuthProxy web root and bundled into the published image, instead of being written to a stale Ingress path and dropped.

Changed

  • The publish pipeline now fails loudly if the frontend build output is empty, instead of silently shipping an image without the frontend.

The Vite build still targeted ../Ingress/wwwroot, a project that no longer
exists after the rename to AuthProxy, so npm run build never populated
Source/AuthProxy/wwwroot. The frontend was therefore absent from the published
image, and the publish pipeline's frontend artifact was silently empty
(upload-artifact defaults to if-no-files-found: warn), which broke the
downstream dotnet-publish download.

- Point the Vite outDir at ../AuthProxy/wwwroot so the frontend builds into
  the AuthProxy web root and is bundled by dotnet publish.
- Set if-no-files-found: error on the frontend artifact upload so an empty
  build output fails loudly at the source instead of failing cryptically
  downstream.

Verified locally: npm run build now writes index.html + assets into
Source/AuthProxy/wwwroot and creates no Ingress directory.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@woksin woksin added the patch label Jul 14, 2026
@woksin
woksin merged commit 70b8e2f into main Jul 14, 2026
3 checks passed
@woksin
woksin deleted the fix/frontend-build-output-path branch July 14, 2026 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant