Client does not update object on response failure

Description

var resp client.Response
// finds results
err := client.Search(&resp, cfg, "node-xray", svc, "^package.json", 0, false, false)
// does not find results
err := client.Search(&resp, cfg, "http-shutdown", svc, "^package.json", 0, false, false)
// Expected Behavior - resp holds results from second call.
// Actual Behavior - resp still holding results from first call.

Guia do colaborador

Pilha de tecnologia

go

Domain

backendapisearch

Tipo Issue

bug

DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.

3

Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.

1-3 hours

Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.

stale

ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.

clear

Prerequisites

Go basicsHTTP client handling

Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.

65

Direção de pesquisa

Examine the client.Search function in the Go client code, likely in a file like client.go. The bug is that when Search fails (returns an error), the resp object is not updated and retains the previous result. Look for early returns on error that skip updating resp. A fix should ensure resp is modified or reset even on failure, or the function should not modify resp on error but the caller expects it to hold new results only on success. Check the current implementation to understand the intended behavior and apply a minimal fix.