Skip to content

Fix update check 500 by removing Server-Name and routing via local proxy#672

Open
cute-omega with Copilot wants to merge 1 commit into
masterfrom
copilot/debug-internal-server-error
Open

Fix update check 500 by removing Server-Name and routing via local proxy#672
cute-omega with Copilot wants to merge 1 commit into
masterfrom
copilot/debug-internal-server-error

Conversation

Copilot AI commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes update-check failures that showed Internal Server Error, code: 500.

Changes

  • Removed the non-standard Server-Name header from the GitHub releases request.
  • Added a request options builder for update checks.
  • When dev-sidecar proxy is enabled, update-check requests now explicitly use local proxy (server.host:server.port) so they are handled by dev-sidecar.
  • Added standard GitHub Accept header and a 10s timeout for update-check requests.

Validation

  • Secret scanning: passed.
  • CodeQL: passed.
  • Code Review tool: failed due to tool-side 400 model error (retried, same result).
  • pnpm install / pnpm lint / package tests could not run in this environment due to registry fetch failure:
    • GET https://registry.npmmirror.com/pnpm: fetch failed

Copilot AI requested a review from cute-omega July 13, 2026 12:48
@cute-omega
cute-omega marked this pull request as ready for review July 13, 2026 12:51
Copilot AI review requested due to automatic review settings July 13, 2026 12:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@cute-omega

Copy link
Copy Markdown
Collaborator

不行,检查更新时仍然报告相同错误

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants