Skip to content

Adjust GitHub Accept header handling for file domains#3

Merged
vansour merged 1 commit intomainfrom
codex/modify-apply_github_headers-signature
Oct 23, 2025
Merged

Adjust GitHub Accept header handling for file domains#3
vansour merged 1 commit intomainfrom
codex/modify-apply_github_headers-signature

Conversation

@vansour
Copy link
Copy Markdown
Owner

@vansour vansour commented Oct 23, 2025

Summary

  • allow apply_github_headers to inspect the target URI and only apply the GitHub API Accept header for API hosts
  • preserve client Accept headers for GitHub file domains by defaulting to */* when none is provided
  • pass the active request URI into apply_github_headers, including during redirect handling

Testing

  • cargo build

https://chatgpt.com/codex/tasks/task_e_68f98d62574883239046a05954f4ff12

@vansour vansour merged commit aaf4f77 into main Oct 23, 2025
@vansour vansour deleted the codex/modify-apply_github_headers-signature branch October 23, 2025 02:36
vansour added a commit that referenced this pull request Dec 7, 2025
…signature

Adjust GitHub Accept header handling for file domains
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