Skip to content

ci: Use basic auth header for per-crash git fetch#49394

Closed
morgankrey wants to merge 2 commits into
mainfrom
fix/background-agent-mvp-git-basic-auth
Closed

ci: Use basic auth header for per-crash git fetch#49394
morgankrey wants to merge 2 commits into
mainfrom
fix/background-agent-mvp-git-basic-auth

Conversation

@morgankrey
Copy link
Copy Markdown
Contributor

@morgankrey morgankrey commented Feb 17, 2026

Summary

  • switch per-crash git auth header from bearer to basic using a base64-encoded x-access-token:$GH_TOKEN value
  • keep existing per-candidate fetch/checkout guards

Why

The latest background_agent_mvp run repeatedly skipped every crash with:

fatal: could not read Username for 'https://github.com': No such device or address

This happened because the workflow was configuring a bearer header for git HTTPS operations; git over GitHub expects basic auth semantics for token-based access.

Validation

Use a base64 encoded x-access-token header for git operations in the background-agent loop so origin fetch works reliably under GH_TOKEN.

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Feb 17, 2026
@zed-community-bot zed-community-bot Bot added the staff Pull requests authored by a current member of Zed staff label Feb 17, 2026
@zed-industries-bot
Copy link
Copy Markdown
Contributor

zed-industries-bot commented Feb 17, 2026

Warnings
⚠️

This PR is missing release notes.

Please add a "Release Notes" section that describes the change:

Release Notes:

- Added/Fixed/Improved ...

If your change is not user-facing, you can use "N/A" for the entry:

Release Notes:

- N/A

Generated by 🚫 dangerJS against b1302e8

@maxdeviant maxdeviant changed the title ci: use basic auth header for per-crash git fetch ci: Use basic auth header for per-crash git fetch Feb 18, 2026
@morgankrey morgankrey closed this Feb 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants