Skip to content

feat: add requires_auth flag for custom providers without authentication#6705

Merged
DOsinga merged 1 commit intoblock:mainfrom
rabi:6693
Jan 28, 2026
Merged

feat: add requires_auth flag for custom providers without authentication#6705
DOsinga merged 1 commit intoblock:mainfrom
rabi:6693

Conversation

@rabi
Copy link
Contributor

@rabi rabi commented Jan 26, 2026

Summary

  • Add AuthMethod::NoAuth variant for providers that don't need auth headers
  • Add requires_auth field to DeclarativeProviderConfig
  • Update CLI to prompt whether provider requires authentication
  • Update UI CustomProviderForm with authentication toggle
  • Refactor create_custom_provider to use CreateCustomProviderParams struct

This enables using local OpenAI-compatible servers (e.g., vLLM, Ollama) that may not require API keys without getting 401 Unauthorized errors.

Type of Change

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

AI Assistance

  • This PR was created or reviewed with AI assistance

Testing

Tested locally

Closes: #6693

- Add AuthMethod::NoAuth variant for providers that don't need auth headers
- Add requires_auth field to DeclarativeProviderConfig
- Update CLI to prompt whether provider requires authentication
- Update UI CustomProviderForm with authentication toggle
- Refactor create_custom_provider to use CreateCustomProviderParams struct

This enables using local OpenAI-compatible servers (e.g., vLLM, Ollama)
that don't require API keys without getting 401 Unauthorized errors.

Change-Id: Ide285de354415eb98dd4e818b7f7b3283756fe88
Signed-off-by: rabi <ramishra@redhat.com>
Copy link
Collaborator

@DOsinga DOsinga left a comment

Choose a reason for hiding this comment

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

this is great!

@DOsinga DOsinga merged commit 2673e8a into block:main Jan 28, 2026
17 checks passed
lifeizhou-ap added a commit that referenced this pull request Jan 28, 2026
* main: (47 commits)
  Upgrade error handling (#6747)
  Fix/filter audience 6703 local (#6773)
  chore: re-sync package-lock.json (#6783)
  upgrade electron to 39.3.0 (#6779)
  allow skipping providers in test_providers.sh (#6778)
  fix: enable custom model entry for OpenRouter provider (#6761)
  Remove codex skills flag support (#6775)
  Improve mcp test (#6671)
  Feat/anthropic custom headers (#6774)
  Fix/GitHub copilot error handling 5845 (#6771)
  fix(ui): respect width parameter in MCP app size-changed notifications (#6376)
  fix: address compilation issue in main (#6776)
  Upgrade GitHub Actions for Node 24 compatibility (#6699)
  fix(google): preserve thought signatures in streaming responses (#6708)
  added reduce motion support for css animations and streaming text (#6551)
  fix: Re-enable subagents for Gemini models (#6513)
  fix(google): use parametersJsonSchema for full JSON Schema support (#6555)
  fix: respect GOOSE_CLI_MIN_PRIORITY for shell streaming output (#6558)
  feat: add requires_auth flag for custom providers without authentication (#6705)
  fix: normalize extension names consistently in ExtensionManager (#6529)
  ...
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.

Goose always sends Authorization header to OpenAI-compatible local servers (causes 401)

2 participants