Skip to content

Fix/configure add provider custom headers#7157

Merged
Abhijay007 merged 3 commits intoblock:mainfrom
Sahil8383:fix/configure-add-provider-custom-headers
Feb 12, 2026
Merged

Fix/configure add provider custom headers#7157
Abhijay007 merged 3 commits intoblock:mainfrom
Sahil8383:fix/configure-add-provider-custom-headers

Conversation

@Sahil8383
Copy link
Contributor

Previously custom headers were only prompted for openai_compatible providers; now they are offered for every provider type.

Summary

The CLI's custom provider configuration only asked for custom headers when adding an OpenAI compatible provider. The backend already supports custom headers for all three provider types (OpenAI, Anthropic, and Ollama compatible). This change updates the configure flow so custom headers are prompted for every provider type.

Type of Change

  • Bug fix
  • Feature
  • Refactor / Code quality
  • Performance improvement
  • Documentation
  • Tests
  • Security fix
  • Build / Release
  • Other (specify below)

Testing

Manual testing: ran goose configure → Custom Providers → Add A Custom Provider, chose Anthropic Compatible and Ollama Compatible, and confirmed the custom headers prompt appears for both (same as for OpenAI Compatible).

Related Issues

Relates to #7014

Screenshots/Demos (for UX changes)

Before:
image

After:
image

Previously custom headers were only prompted for openai_compatible
providers; now they are offered for every provider type.

Signed-off-by: Sahil8383 <sahilbhatt915@gmail.com>
Signed-off-by: Sahil8383 <sahilbhatt915@gmail.com>
@Sahil8383
Copy link
Contributor Author

@dianed-square could you take a look at this PR when you have a moment? (Fixes: #7014 )

@dianed-square
Copy link
Contributor

@Sahil8383 Thanks and looks good to me and goose, but I'll defer approval to the eng team.
But you can probably remove the comment on line 1913 now that all types are supported.

Signed-off-by: Sahil8383 <sahilbhatt915@gmail.com>
@Sahil8383
Copy link
Contributor Author

Sahil8383 commented Feb 12, 2026

Hello @DOsinga and @katzdave,
Could you have a loot at this PR?

Copy link
Collaborator

@Abhijay007 Abhijay007 left a comment

Choose a reason for hiding this comment

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

LGTM

@Abhijay007 Abhijay007 added this pull request to the merge queue Feb 12, 2026
Merged via the queue into block:main with commit d1aa571 Feb 12, 2026
19 checks passed
tlongwell-block added a commit that referenced this pull request Feb 12, 2026
…provenance

* origin/main: (68 commits)
  Upgraded npm packages for latest security updates (#7183)
  docs: reasoning effort levels for Codex provider (#6798)
  Fix speech local (#7181)
  chore: add .gooseignore to .gitignore (#6826)
  Improve error message logging from electron (#7130)
  chore(deps): bump jsonwebtoken from 9.3.1 to 10.3.0 (#6924)
  docs: standalone mcp apps and apps extension (#6791)
  workflow: auto-update cli-commands on release (#6755)
  feat(apps): Integrate AppRenderer from @mcp-ui/client SDK (#7013)
  fix(MCP): decode resource content (#7155)
  feat: reasoning_content in API for reasoning models (#6322)
  Fix/configure add provider custom headers (#7157)
  fix: handle keyring fallback as success (#7177)
  Update process-wrap to 9.0.3 (9.0.2 is yanked) (#7176)
  feat: support extra field in chatcompletion tool_calls for gemini openai compat (#6184)
  fix: replace panic with proper error handling in get_tokenizer (#7175)
  Lifei/smoke test for developer (#7174)
  fix text editor view broken (#7167)
  docs: White label guide (#6857)
  Add PATH detection back to developer extension (#7161)
  ...

# Conflicts:
#	.github/workflows/nightly.yml
jh-block added a commit that referenced this pull request Feb 13, 2026
* origin/main: (21 commits)
  nit: show dir in title, and less... jank (#7138)
  feat(gemini-cli): use stream-json output and re-use session (#7118)
  chore(deps): bump qs from 6.14.1 to 6.14.2 in /documentation (#7191)
  Switch jsonwebtoken to use aws-lc-rs (already used by rustls) (#7189)
  chore(deps): bump qs from 6.14.1 to 6.14.2 in /evals/open-model-gym/mcp-harness (#7184)
  Add SLSA build provenance attestations to release workflows (#7097)
  fix save and run recipe not working (#7186)
  Upgraded npm packages for latest security updates (#7183)
  docs: reasoning effort levels for Codex provider (#6798)
  Fix speech local (#7181)
  chore: add .gooseignore to .gitignore (#6826)
  Improve error message logging from electron (#7130)
  chore(deps): bump jsonwebtoken from 9.3.1 to 10.3.0 (#6924)
  docs: standalone mcp apps and apps extension (#6791)
  workflow: auto-update cli-commands on release (#6755)
  feat(apps): Integrate AppRenderer from @mcp-ui/client SDK (#7013)
  fix(MCP): decode resource content (#7155)
  feat: reasoning_content in API for reasoning models (#6322)
  Fix/configure add provider custom headers (#7157)
  fix: handle keyring fallback as success (#7177)
  ...
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