Skip to content

Fix 404 on HEAD for the FW Lite App Installer manifest - #2596

Merged
hahn-kev merged 1 commit into
developfrom
fwlite-appinstaller-head
Aug 24, 2026
Merged

Fix 404 on HEAD for the FW Lite App Installer manifest#2596
hahn-kev merged 1 commit into
developfrom
fwlite-appinstaller-head

Conversation

@hahn-kev-bot

@hahn-kev-bot hahn-kev-bot commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

when windows tries to autoupdate from an AppInstaller it calls HEAD on the url, this currently fails with a 404. This fixes that.

AI summary

Windows App Installer sends a HEAD request to the .appinstaller manifest URL and requires a Content-Length header. ASP.NET Core routing does not match HEAD against an [HttpGet]-only action, so the router returned 405.

This adds [HttpHead] on the same AppInstaller action. No separate handler: File() already sets Content-Length and Content-Type and omits the body for HEAD. Other fwlite-release endpoints are JSON or redirects and are not probed by App Installer.

Test plan

  • AppInstallerActionAcceptsHeadOnTheSamePathAsGet passes
  • curl -I https://<host>/api/fwlite-release/FieldWorksLite.appinstaller returns 200 with Content-Length and Content-Type: application/appinstaller
  • GET of the same URL still returns the XML body

Made with Cursor

Windows App Installer probes the .appinstaller URL with HEAD and requires Content-Length; [HttpGet] alone 405s that request.

Co-authored-by: Cursor <cursoragent@cursor.com>
@coderabbitai

coderabbitai Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 45432262-1537-4c86-8041-9805f3124759


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added the 📦 Lexbox issues related to any server side code, fw-headless included label Aug 24, 2026
@hahn-kev hahn-kev changed the title Fix 405 on HEAD for the FW Lite App Installer manifest Fix 404 on HEAD for the FW Lite App Installer manifest Aug 24, 2026
@hahn-kev hahn-kev added the self-reviewed 👁️ I reviewed this myself and with AI and decided it was safe to merge without a second set of eyes label Aug 24, 2026
@hahn-kev
hahn-kev merged commit 21d82b6 into develop Aug 24, 2026
12 checks passed
@hahn-kev
hahn-kev deleted the fwlite-appinstaller-head branch August 24, 2026 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📦 Lexbox issues related to any server side code, fw-headless included self-reviewed 👁️ I reviewed this myself and with AI and decided it was safe to merge without a second set of eyes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants