Skip to content

fix(search): skip catalog-default SearXNG localhost:8888 - #10981

Merged
diegosouzapw merged 1 commit into
diegosouzapw:release/v3.8.50from
RaviTharuma:fix/10976-skip-default-searxng
Aug 21, 2026
Merged

fix(search): skip catalog-default SearXNG localhost:8888#10981
diegosouzapw merged 1 commit into
diegosouzapw:release/v3.8.50from
RaviTharuma:fix/10976-skip-default-searxng

Conversation

@RaviTharuma

@RaviTharuma RaviTharuma commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Catalog default searxng-search is http://localhost:8888/search. In Docker/K8s that port is closed.
  • Combined with OMNIROUTE_ALLOW_PRIVATE_PROVIDER_URLS=true (required for ClusterIP providers), every search ECONNREFUSEDs :8888, logs a 502, then burns the next provider (Brave free-tier 429).
  • Skip that catalog default unless baseUrl was overridden. Last-resort fallback can then use duckduckgo-free.

Related Issues

Validation

  • Change type: provider / routing / other (search)
  • Focused tests: node --import tsx/esm --test tests/unit/searxng-loopback-default.test.ts (3/3)
  • npm run lint (CI)
  • Branched from origin/release/v3.8.50
  • New automated test in this PR

Tests Added Or Updated

  • tests/unit/searxng-loopback-default.test.ts

Coverage Notes

  • Helper isUnconfiguredLoopbackSearchProvider is unit-tested. handleSearch skip path and /v1/search last-resort continue are thin callers of that helper.

Reviewer Notes

  • Operators who actually run SearXNG on :8888 still can: change is only the catalog default URL. A custom baseUrl is attempted as before.

@RaviTharuma
fix(search): skip catalog-default SearXNG localhost:8888
2e3c2a6 
Docker/K8s has no SearXNG on :8888. The catalog default plus
OMNIROUTE_ALLOW_PRIVATE_PROVIDER_URLS made every search
ECONNREFUSED, 502, then burn the next provider (Brave 429).

Skip http://localhost:8888/search unless the operator overrides
baseUrl. Last-resort fallback then reaches duckduckgo-free.

Closes #10976.
@RaviTharuma RaviTharuma mentioned this pull request Aug 21, 2026
Closed
@diegosouzapw
diegosouzapw merged commit 143fd78 into diegosouzapw:release/v3.8.50 Aug 21, 2026
7 of 16 checks passed
diegosouzapw pushed a commit that referenced this pull request Aug 21, 2026
@RaviTharuma
feat(search): first-class X Search via SuperGrok x_search (#10988)
137e49e 
⭐5 — Provider x-search de primeira classe (SuperGrok/xAI x_search) em POST /v1/search e MCP omniroute_x_search. Fallback de credenciais xai-oauth→xao→xai; distinto de web search e do X Developer MCP.

Reconciliado com o release tip (que já incluía #10981 "skip catalog-default SearXNG" deste mesmo lote): merge trouxe 5 conflitos reais de contagem gerada (llm.txt/README.md/AGENTS.md/PROVIDER_REFERENCE.md/SVGs/46 mirrors i18n, todos verificados como bump puro 347→348, sem perda de conteúdo do HEAD) + 1 conflito real de mergeable=CONFLICTING.

Durante a validação, os 3 testes novos de SearXNG expuseram um bug real de interação com #10981: `isUnconfiguredLoopbackSearchProvider()` checava o baseUrl ESTÁTICO do catálogo em vez do baseUrl efetivo (após override de `provider_options.baseUrl` ou `providerSpecificData.baseUrl` da conexão), então QUALQUER request a searxng-search — mesmo com override customizado — era rejeitado como se fosse o default não-configurado. Corrigido em `open-sse/handlers/search.ts` (resolve o baseUrl efetivo via `resolveSearchBaseUrl()` antes do skip-check, tanto para o provider primário quanto o alternate). Um teste do próprio #10988 que assumia o comportamento pré-#10981 (default localhost:8888 sempre atendido) foi atualizado para refletir o comportamento já mesclado e intencional (503 quando não configurado).

Validação completa: typecheck limpo, 70/70 testes unit (search-route/search-registry/x-search-provider/searxng-loopback-default), 24/24 vitest MCP, 14/14 integration (search-providers-catalog), lint limpo nos arquivos tocados, docs-counts-sync OK (2 drifts soft pré-existentes, não relacionados), gates estáticos (file-size/complexity/cognitive/dead-code/changelog) todos OK.
Sa3id23 pushed a commit to Sa3id23/OmniRoute that referenced this pull request Aug 21, 2026
@RaviTharuma @Sa3id23
fix(search): skip catalog-default SearXNG localhost:8888 (diegosouzap…
ff47fd5 
…w#10981)

⭐5 — Default do catálogo searxng-search (localhost:8888) sempre falha em Docker/K8s, queima o próximo provider (Brave free-tier 429). Agora pula o default não-configurado a menos que baseUrl tenha sido sobrescrito; fallback duckduckgo-free continua disponível. 3/3 testes. Fecha diegosouzapw#10976.
Sa3id23 pushed a commit to Sa3id23/OmniRoute that referenced this pull request Aug 21, 2026
@RaviTharuma @Sa3id23
feat(search): first-class X Search via SuperGrok x_search (diegosouza…
fdda33f 
…pw#10988)

⭐5 — Provider x-search de primeira classe (SuperGrok/xAI x_search) em POST /v1/search e MCP omniroute_x_search. Fallback de credenciais xai-oauth→xao→xai; distinto de web search e do X Developer MCP.

Reconciliado com o release tip (que já incluía diegosouzapw#10981 "skip catalog-default SearXNG" deste mesmo lote): merge trouxe 5 conflitos reais de contagem gerada (llm.txt/README.md/AGENTS.md/PROVIDER_REFERENCE.md/SVGs/46 mirrors i18n, todos verificados como bump puro 347→348, sem perda de conteúdo do HEAD) + 1 conflito real de mergeable=CONFLICTING.

Durante a validação, os 3 testes novos de SearXNG expuseram um bug real de interação com diegosouzapw#10981: `isUnconfiguredLoopbackSearchProvider()` checava o baseUrl ESTÁTICO do catálogo em vez do baseUrl efetivo (após override de `provider_options.baseUrl` ou `providerSpecificData.baseUrl` da conexão), então QUALQUER request a searxng-search — mesmo com override customizado — era rejeitado como se fosse o default não-configurado. Corrigido em `open-sse/handlers/search.ts` (resolve o baseUrl efetivo via `resolveSearchBaseUrl()` antes do skip-check, tanto para o provider primário quanto o alternate). Um teste do próprio diegosouzapw#10988 que assumia o comportamento pré-diegosouzapw#10981 (default localhost:8888 sempre atendido) foi atualizado para refletir o comportamento já mesclado e intencional (503 quando não configurado).

Validação completa: typecheck limpo, 70/70 testes unit (search-route/search-registry/x-search-provider/searxng-loopback-default), 24/24 vitest MCP, 14/14 integration (search-providers-catalog), lint limpo nos arquivos tocados, docs-counts-sync OK (2 drifts soft pré-existentes, não relacionados), gates estáticos (file-size/complexity/cognitive/dead-code/changelog) todos OK.
@RaviTharuma
RaviTharuma deleted the fix/10976-skip-default-searxng branch September 23, 2026 19:37
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
@RaviTharuma
fix(search): skip catalog-default SearXNG localhost:8888 (diegosouzap…
ca3b297 
…w#10981)

⭐5 — Default do catálogo searxng-search (localhost:8888) sempre falha em Docker/K8s, queima o próximo provider (Brave free-tier 429). Agora pula o default não-configurado a menos que baseUrl tenha sido sobrescrito; fallback duckduckgo-free continua disponível. 3/3 testes. Fecha diegosouzapw#10976.
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
@RaviTharuma
feat(search): first-class X Search via SuperGrok x_search (diegosouza…
ee76ce7 
…pw#10988)

⭐5 — Provider x-search de primeira classe (SuperGrok/xAI x_search) em POST /v1/search e MCP omniroute_x_search. Fallback de credenciais xai-oauth→xao→xai; distinto de web search e do X Developer MCP.

Reconciliado com o release tip (que já incluía diegosouzapw#10981 "skip catalog-default SearXNG" deste mesmo lote): merge trouxe 5 conflitos reais de contagem gerada (llm.txt/README.md/AGENTS.md/PROVIDER_REFERENCE.md/SVGs/46 mirrors i18n, todos verificados como bump puro 347→348, sem perda de conteúdo do HEAD) + 1 conflito real de mergeable=CONFLICTING.

Durante a validação, os 3 testes novos de SearXNG expuseram um bug real de interação com diegosouzapw#10981: `isUnconfiguredLoopbackSearchProvider()` checava o baseUrl ESTÁTICO do catálogo em vez do baseUrl efetivo (após override de `provider_options.baseUrl` ou `providerSpecificData.baseUrl` da conexão), então QUALQUER request a searxng-search — mesmo com override customizado — era rejeitado como se fosse o default não-configurado. Corrigido em `open-sse/handlers/search.ts` (resolve o baseUrl efetivo via `resolveSearchBaseUrl()` antes do skip-check, tanto para o provider primário quanto o alternate). Um teste do próprio diegosouzapw#10988 que assumia o comportamento pré-diegosouzapw#10981 (default localhost:8888 sempre atendido) foi atualizado para refletir o comportamento já mesclado e intencional (503 quando não configurado).

Validação completa: typecheck limpo, 70/70 testes unit (search-route/search-registry/x-search-provider/searxng-loopback-default), 24/24 vitest MCP, 14/14 integration (search-providers-catalog), lint limpo nos arquivos tocados, docs-counts-sync OK (2 drifts soft pré-existentes, não relacionados), gates estáticos (file-size/complexity/cognitive/dead-code/changelog) todos OK.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@diegosouzapw diegosouzapw Awaiting requested review from diegosouzapw diegosouzapw is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[BUG] Default SearXNG http://localhost:8888 is probed in Docker/K8s and 502s every search

2 participants

@RaviTharuma @diegosouzapw