gooneyryan/hermes-webui
1
0
Fork 0
mirror of https://github.com/nesquena/hermes-webui.git synced 2026-05-29 21:20:31 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: skip get_auth_status() fallback for known API-key providers

Browse Source 
Avoids unnecessary latency on the Settings page by restricting the
OAuth auth-status fallback to providers that are not in _PROVIDER_ENV_VAR.

Review feedback (PR #1221): the get_auth_status() call in the else branch
was firing for every unconfigured API-key provider (openai, anthropic, etc.),
adding a network round-trip per provider. Now it only runs for providers
that are not known API-key providers (custom/OAuth-capable providers).
This commit is contained in:
bergeouss
2026-04-28 10:58:23 +00:00
committed by Hermes Agent
parent ae2ed1a4e7
commit 0741a2ab9f
1 changed files with 11 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
api/providers.py
+11 -6
View File
@@ -312,12 +312,17 @@ def get_providers() -> dict[str, Any]:
key_source = "config_yaml"
else:
key_source = "config_yaml"
else:
# Fallback: provider may be authenticated via hermes auth even
# though it is not in the hardcoded _OAUTH_PROVIDERS set
# (e.g. Anthropic connected via OAuth). Check live auth
# status so the Providers tab agrees with the model picker
# (#1212).
elif pid not in _PROVIDER_ENV_VAR:
# Fallback: provider is not a known API-key provider and not in
# the hardcoded _OAUTH_PROVIDERS set. It may be a custom or
# newly-added OAuth provider (e.g. Anthropic connected via OAuth).
# Check live auth status so the Providers tab agrees with the
# model picker (#1212).
#
# IMPORTANT: we skip providers in _PROVIDER_ENV_VAR because they
# are pure API-key providers — calling get_auth_status() for every
# unconfigured API-key provider would add unnecessary latency
# (network round-trip per provider) on the Settings page.
try:
from hermes_cli.auth import get_auth_status as _gas
status = _gas(pid)